Remove decimal places handling and unused methods from textbox[type=number]

RESOLVED FIXED in Firefox 60

Status

()

RESOLVED FIXED
a year ago
8 months ago

People

(Reporter: ntim, Assigned: ntim)

Tracking

(Blocks: 1 bug)

unspecified
mozilla60
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox60 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

a year ago
As seen in bug 1429573, the following can be removed from the numberbox binding: decimal places handling, increase() and decrease() methods.
(Assignee)

Updated

a year ago
Summary: Remove unused features from textbox[type=number] → Remove decimal places handling and unused methods from textbox[type=number]
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Comment 4

a year ago
ni? Paenglab about potential comm-central changes.
Flags: needinfo?(richard.marti)
Thank you for the info. I found only one place where we use decimal places: https://dxr.mozilla.org/comm-central/source/mail/components/preferences/notifications.xul#40

This should be easy to change.
Flags: needinfo?(richard.marti)
Comment hidden (mozreview-request)
Attachment #8941671 - Flags: review?(dao+bmo) → review?(enndeakin)
(Assignee)

Comment 7

a year ago
Neil, are you able to review this ?
Flags: needinfo?(enndeakin)
(Assignee)

Updated

a year ago
Attachment #8941671 - Flags: review?(enndeakin) → review?(paolo.mozmail)

Comment 8

a year ago
mozreview-review
Comment on attachment 8941671 [details]
Bug 1429625 - Remove decimal places handling and unused methods from numberbox binding.

https://reviewboard.mozilla.org/r/211916/#review220476

Thanks!
Attachment #8941671 - Flags: review?(paolo.mozmail) → review+
Comment hidden (mozreview-request)
(Assignee)

Updated

a year ago
Flags: needinfo?(enndeakin)

Comment 10

a year ago
Pushed by ntim.bugs@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/5beaa39e3c58
Remove decimal places handling and unused methods from numberbox binding. r=Paolo

Comment 11

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/5beaa39e3c58
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox60: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
(Assignee)

Updated

a year ago
Depends on: 1432462
(Assignee)

Updated

a year ago
Blocks: 1429573
this was useful for my restartless Addon MenuOnTop (setting border radius in multiples of em). Now it's broken so I need to change to px. why remove something so useful?
You need to log in before you can comment on or make changes to this bug.