Remove the "number-control.css" UA sheet

RESOLVED FIXED in Firefox 61

Status

()

Core
Layout: Form Controls
--
minor
RESOLVED FIXED
a year ago
11 days ago

People

(Reporter: mats, Assigned: heycam)

Tracking

({perf})

Trunk
mozilla61
Points:
---

Firefox Tracking Flags

(firefox53 affected, firefox61 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

a year ago
I think we can move this rule:
http://searchfox.org/mozilla-central/source/layout/style/res/number-control.css
inside a @supports -moz-bool-pref("dom.forms.number") {} clause
in forms.css instead, and revert the other code changes from bug 974430.

(Or we could just remove the dom.forms.number pref altogether since it's
been enabled by default for a few years now.  I filed bug 1317539 for that.)
Comment hidden (mozreview-request)
(Assignee)

Updated

12 days ago
Assignee: nobody → cam
Status: NEW → ASSIGNED
(Reporter)

Comment 2

12 days ago
mozreview-review
Comment on attachment 8966444 [details]
Bug 1317541 - Merge number-control.css back into forms.css.

https://reviewboard.mozilla.org/r/235152/#review241170
Attachment #8966444 - Flags: review?(mats) → review+

Comment 3

11 days ago
Pushed by cam@mcc.id.au:
https://hg.mozilla.org/integration/autoland/rev/7d516d8ca975
Merge number-control.css back into forms.css. r=mats

Comment 4

11 days ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/7d516d8ca975
Status: ASSIGNED → RESOLVED
Last Resolved: 11 days ago
status-firefox61: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.