Jetpack: Admin Page: improve checkbox status in Firefox

Created on 7 Mar 2017  Â·  8Comments  Â·  Source: Automattic/jetpack

The checkboxes displayed in Jetpack Admin Settings display differently depending on your browser:

Here is how it looks in Chrome Version 56.0.2924.87 (64-bit), on Mac:

screen shot 2017-03-06 at 22 39 52

And in Firefox 51.0.1 (64-bit), on Mac:

screen shot 2017-03-06 at 22 40 59

In Firefox, the checkboxes look a lot better when I remove a -moz-appearance: none; rule:

screen shot 2017-03-06 at 22 49 29

It would be nice to update the checkboxes to get a similar look across browsers.

Admin Page [Status] Needs Design Review [Status] Stale [Type] Enhancement

Most helpful comment

It won't be an issue with checkboxes... because we no longer have checkboxes! :D But it's now an issue with radio buttons.
The reason (for both radio buttons and checkboxes) is that WP core uses :before to add the styling, which works fine in Chrome

captura de pantalla 2017-03-23 a las 10 24 44

but not in Firefox

captura de pantalla 2017-03-23 a las 10 24 38

and that's because Firefox adheres to the standard and doesn't allow :before in input elements since they're not a container of other elements like span, div or p are.

Adding -moz-appearance: none; in this case actually makes them look uglier:

captura de pantalla 2017-03-23 a las 10 35 13

All 8 comments

FYI the issue I originally reported was on Windows. Realized I never said that. :-(

@singerb Could you post screenshots of how things look like for you on Windows? Thanks!

Here we go!

Chrome, looks fine:

chrome-settings

Firefox, checkbox checked but nothing showing:

firefox-bad-settings

Firefox, checkbox checked and -moz-appearance: none removed:

firefox-mod-css-settings

Will the current settings work will make this issue obsolete? Cc @MichaelArestad @eliorivero

It won't be an issue with checkboxes... because we no longer have checkboxes! :D But it's now an issue with radio buttons.
The reason (for both radio buttons and checkboxes) is that WP core uses :before to add the styling, which works fine in Chrome

captura de pantalla 2017-03-23 a las 10 24 44

but not in Firefox

captura de pantalla 2017-03-23 a las 10 24 38

and that's because Firefox adheres to the standard and doesn't allow :before in input elements since they're not a container of other elements like span, div or p are.

Adding -moz-appearance: none; in this case actually makes them look uglier:

captura de pantalla 2017-03-23 a las 10 35 13

@eliorivero nice. I think we can live this for now since it matches core behavior, but might be a nice future improvement.

This issue has been marked as stale. This happened because:

  • It has been inactive in the past 6 months.
  • It hasn’t been labeled `[Pri] Blocker`, `[Pri] High`.

No further action is needed. But it's worth checking if this ticket has clear reproduction steps and it is still reproducible. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation.

Closing this for now because of the lack of activity on this. We can always reopen in the future if needed.

Was this page helpful?
0 / 5 - 0 ratings