Opened 12 years ago
Closed 12 years ago
#21611 closed defect (bug) (fixed)
Remove CSS hover styling of postboxes with disabled JS
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 3.5 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Administration | Keywords: | has-patch commit |
| Focuses: | Cc: |
Description
If JS is disabled in the browser, post metaboxes can neither be moved/sorted nor expanded/shrunken.
However, due to the current wp-admin-CSS, the boxes still show effects on mouse hover (like text color change, other mouse pointers, visible arrow handle).
This can confuse users as nothing is happening when clicking on them, so I'd say that this is a (though rare to see) styling bug.
I suggest to simply only do these hover effects, if JS is enabled.
This is as simple as prepending the relevant CSS selectors with the ".js" class that we have (or not have) on <body>.
The attached patch does this for post metaboxes, as well as for the widgets page, where the same occurs under a different name.
Attachments (2)
Change History (16)
#3
@
12 years ago
- Component changed from UI to Administration
- Keywords needs-refresh added; ui-feedback removed
- Milestone changed from Awaiting Review to Future Release
#4
@
12 years ago
- Keywords 3.6-early added
Great :-)
I will gladly refresh the patch for the next dev cycle, as I assume that this comes to late for 3.5.
#6
@
12 years ago
- Keywords needs-refresh 3.6-early removed
Ah, I had misinterpreted you :-) I had thought that there are more changes necessary, due to recent CSS changes in core. Turned out that all that was needed was the switch to the new CSS file names.
So, here's the refreshed patch.
#7
@
12 years ago
- Keywords commit added
- Milestone changed from Future Release to 3.5
This is so much better for no JS - I'd consider current behavior buggy looking and feeling. Makes perfect sense - let's do it.
#8
@
12 years ago
- Owner set to nacin
- Resolution set to fixed
- Status changed from new to closed
In 22391:
#9
@
12 years ago
Still showing on Retina displays. Patch forthcoming this evening if noone gets to it first.
#10
@
12 years ago
- Keywords needs-patch added; has-patch commit removed
- Resolution fixed deleted
- Status changed from closed to reopened
Seems we need to add that same selector to the media query blocks where applicable for HiDPI, even as unlikely as retina + no-JS seems.
Makes sense to me.