non-native theme input placeholder text padding does not match regular text padding
Categories
(Core :: Widget, defect)
Tracking
()
People
(Reporter: heycam, Unassigned)
References
Details
Attachments
(1 file)
(deleted),
image/png
|
Details |
Some reftests fail with the non-native theme enabled because the padding applied to an input's placeholder text is not the same as that applied to the regular text (the editor div). See the attached screen shot for an example.
This seems to be because we don't correctly store the used padding on the scroll frame that contains the placeholder. That decision is made here:
as we've passed in an aPadding
value to manually inherit down the padding value from the nsTextControlFrame
. The editor div's frame is marked as a reflow root here:
which causes us to store the UsedPaddingProperty()
. I'm not sure why we don't store this whenever we have an aPadding
value passed in, since otherwise later calls to GetUsedPadding()
will return values that come from style and be different from what we used to reflow.
Comment 1•4 years ago
|
||
I think bug 1577530 will fix this incidentally. I don't think we want to do it unconditionally because we pass the padding override for all scrolled content and such.
Reporter | ||
Comment 2•4 years ago
|
||
Thanks, confirmed fixed with that patch.
Description
•