If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Remove max-width & max-height from input[type=checkbox] in content.css

RESOLVED FIXED in Firefox 36

Status

()

Firefox for Android
General
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: miketaylr, Assigned: miketaylr)

Tracking

Trunk
Firefox 36
x86
Mac OS X
Points:
---
Bug Flags:
qe-verify -

Firefox Tracking Flags

(firefox35 ?, firefox36 fixed, fennec35+)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

3 years ago
(I don't know if this better fits in the Theme and Visual Design component).

Originally reported in Bug 1078393, the current max-width: 14px style prevents Fennec users from being able to interact with "custom" checkbox styling techniques [1].

I wrote about this in https://miketaylr.com/posts/2014/10/max-width-in-firefox-mobile-browser-checkboxes.html. 

Compare Mobile Safari, Chrome Mobile, Opera Mobile, and Windows Phone with Fennec (or Firefox OS, to be filed) on the linked demo [2].

Note that desktop doesn't have max-width on it's checkboxes so people who don't test on mobile won't even know they need to manually override the ua stylesheet's max-width.

The patch is simple... my only concern is there might be some internal code relying on the max-width of a checkbox being exactly 14px. Weirder things have happened.

[1] http://tympanus.net/codrops/2012/09/13/button-switches-with-checkboxes-and-css3-fanciness/
[2] https://miketaylr.com/bzla/1078393-1.html
(Assignee)

Updated

3 years ago
(Assignee)

Updated

3 years ago
Summary: Remove max-width from input[type=checkbox] in content.css → Remove max-width & max-height from input[type=checkbox] in content.css
(Assignee)

Updated

3 years ago
tracking-fennec: --- → ?
(Assignee)

Comment 1

3 years ago
Created attachment 8505605 [details] [diff] [review]
1082757.patch

Here's a patch. I tried to run reftests locally but ran into Bug 1083279.
(Assignee)

Updated

3 years ago
Component: Graphics, Panning and Zooming → General
Comment on attachment 8505605 [details] [diff] [review]
1082757.patch

Review of attachment 8505605 [details] [diff] [review]:
-----------------------------------------------------------------

I think this seems fine.
Attachment #8505605 - Flags: review?(wjohnston) → review+
(Assignee)

Comment 3

3 years ago
Created attachment 8505622 [details] [diff] [review]
1082757.patch
Attachment #8505605 - Attachment is obsolete: true
(Assignee)

Comment 4

3 years ago
Updated commit message to add "r=wesj"
Keywords: checkin-needed
(Assignee)

Updated

3 years ago
Blocks: 1083536
(Assignee)

Updated

3 years ago
No longer blocks: 1083536
See Also: → bug 1083536
(Assignee)

Updated

3 years ago
Assignee: nobody → miket
https://hg.mozilla.org/integration/fx-team/rev/2316f7e0b196
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/2316f7e0b196
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 36
tracking-fennec: ? → 35+
Now that this is fixed in 36, do we want this for Fennec 35 as well?
status-firefox35: --- → ?
status-firefox36: --- → fixed
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.