Closed Bug 1531343 Opened 2 years ago Closed 2 years ago

Font Editor: Remove auto-increment behavior from range inputs

Categories

(DevTools :: Inspector: Fonts, enhancement, P3)

enhancement

Tracking

(firefox67 fixed)

RESOLVED FIXED
Firefox 67
Tracking Status
firefox67 --- fixed

People

(Reporter: rcaliman, Assigned: rcaliman)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

The feature isn't easily discoverable and when it does work, its user experience isn't good. We never invested time to refine it and we're unlikely to do so. At this point, this is dead code so we should remove it.

What is getting removed is the logic to support the esoteric auto-increment behavior which happens when dragging the range input thumb to the end of the slider and waiting for a while (1000ms). There's needless complexity in handling the concert of events for an experience with little value.

Users can still input out-of-range values in the text input field for the properties which allow it: font-size, line-height.

With @mbalfanz's permission, I am removing the auto-increment behavior from the range inputs in the Font Editor.

The feature isn't easily discoverable and when it does work, its user experience isn't good. We never invested time to refine it and we're unlikely to do so. At this point, this is dead code so we should remove it.

What is getting removed is the logic to support the esoteric auto-increment behavior which happens when dragging the range input thumb to the end of the slider and waiting for a while (1000ms). There's needless complexity in handling the concert of events for a little perceived user value.

Users can still type out-of-range values in the text input field for the properties which allow it: font-size and line-height. For these properties' components, there's a new prop, allowOverflow, which permits values above the max defined. Other properties are bounded by min and max just like before.

Pushed by rcaliman@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6a9be28daf6e
Remove auto-increment from range inputs in Font Editor; r=gl
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
You need to log in before you can comment on or make changes to this bug.