Bug 1546451 Comment 14 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

> I guess it's too late for that now, but this is still an issue.
> De-selecting `height: 3px` from the `.font-value-slider::-moz-range-track` selector doesn't change anything in the spacing slider.

I'm unable to reproduce that difference in the slider thickness, so this needs a deeper investigation, but not a cause for a P1 bug now.
Do you, by any change, have a non-HiDPI monitor that may be causing the difference? I have only HiDPI monitors to test with.

> Also the spacer is now a bit longer than the other two, but this is present in LTR as well.

That's a slight bug caused by the `font-value-label` dimensions used on the letter-spacing as a default keyword. On click on it, or when changing the letter-spacing, it gets replaced with an input + select dropdown which seem to align with the other two for line-height and font-size.
 

> Should I file a new bug for these two issues and CC you?

Yes, please file another bug. No need to CC, it shows up on my radar if it's on the Inspector: Fonts component.
Thanks!
> I guess it's too late for that now, but this is still an issue.
> De-selecting `height: 3px` from the `.font-value-slider::-moz-range-track` selector doesn't change anything in the spacing slider.

I'm unable to reproduce that difference in the slider thickness, so this needs a deeper investigation, but not a cause for a P1 bug now.
Do you, by any chance, have a non-HiDPI monitor that may be causing the difference? I have only HiDPI monitors to test with.

> Also the spacer is now a bit longer than the other two, but this is present in LTR as well.

That's a slight bug caused by the `font-value-label` dimensions used on the letter-spacing as a default keyword. On click on it, or when changing the letter-spacing, it gets replaced with an input + select dropdown which seem to align with the other two for line-height and font-size.
 

> Should I file a new bug for these two issues and CC you?

Yes, please file another bug. No need to CC, it shows up on my radar if it's on the Inspector: Fonts component.
Thanks!

Back to Bug 1546451 Comment 14