Opened on 12/19/2017 at 09:25:55 AM
Closed on 04/05/2018 at 01:04:05 PM
#6206 closed defect (duplicate)
Fix no-js state on acceptableads.com
| Reported by: | ire | Assignee: | |
|---|---|---|---|
| Priority: | P4 | Milestone: | |
| Module: | Websites | Keywords: | goodfirstbug |
| Cc: | juliandoucette, wspee | Blocked By: | |
| Blocking: | Platform: | Unknown / Cross platform | |
| Ready: | yes | Confidential: | no |
| Tester: | Unknown | Verified working: | no |
| Review URL(s): | |||
Description
Environment
Any browser
How to reproduce
- Go to acceptableads.com
- Remove the js class from the <html> element (or change it to no-js)
Observed behaviour
See that the layout is broken, particularly on mobile.
Expected behaviour
- Any style that is only applied to a .js state should also have a workable fallback for .no-js
- The <html> element should, by default have a .no-js class that is changed to .js only if JS is enabled
Attachments (0)
Change History (6)
comment:1 follow-up: ↓ 2 Changed on 01/05/2018 at 01:58:33 PM by juliandoucette
- Keywords goodfirstbug added
- Priority changed from P2 to P3
comment:2 in reply to: ↑ 1 Changed on 01/08/2018 at 09:53:40 AM by ire
Replying to juliandoucette:
nothing is broken for not-js browsers AFAICT
The mobile navigation is not usable without JS
comment:3 Changed on 01/08/2018 at 01:25:27 PM by juliandoucette
The mobile navigation is not usable without JS
Ack. Thank you for pointing this out. I don't think this warrants P2 nonetheless.
comment:4 Changed on 01/25/2018 at 03:01:19 AM by juliandoucette
- Priority changed from P3 to P4
comment:5 Changed on 01/30/2018 at 11:18:16 PM by juliandoucette
- Ready set
comment:6 Changed on 04/05/2018 at 01:04:05 PM by juliandoucette
- Resolution set to duplicate
- Status changed from new to closed
Note: See
TracTickets for help on using
tickets.
Reprioritizing because JavaScript doesn't actually add the js/no-js classes (nothing is broken for not-js browsers AFAICT).