Bug 53632 – [Chromium] Option text in select popup does not align with menulist button text

• Component: New Bugs
• Status: RESOLVED FIXED

[xiyuan]

Created attachment 80978 [details]
Screenshot of problem.

Menulist button has addition paddings from m_innerBlock besides css paddings and PopupListBox should include those paddings as well.

[xiyuan]

Created attachment 80987 [details]
Proposed patch.

[xiyuan]

Comment on attachment 80987 [details]
Proposed patch.

This patch has side effect that because the m_innerBlock padding include the space for scrollbar and PopupListBox::layout might add space for scrollbar again.

[xiyuan]

Comment on attachment 80987 [details]
Proposed patch.

Resurrect the patch as we are protected from counting scrollbar width twice in RenderMenuList::clientPaddingRight which returns fixed endOfLinePadding.

[Tony Chang]

Comment on attachment 80987 [details]
Proposed patch.

View in context: https://bugs.webkit.org/attachment.cgi?id=80987&action=review

> Source/WebCore/rendering/RenderMenuList.cpp:489
> +    return paddingLeft() + m_innerBlock->paddingLeft();

Won't this change the offset for all webkit platforms?  Is that intentional?

[xiyuan]

(In reply to comment #4)
> (From update of attachment 80987 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=80987&action=review
> 
> > Source/WebCore/rendering/RenderMenuList.cpp:489
> > +    return paddingLeft() + m_innerBlock->paddingLeft();
> 
> Won't this change the offset for all webkit platforms?  Is that intentional?

This should affect Windows and Linux. Both platforms have the alignment problem.

Mac probably will not be affected by this because its popup shows up via showExternal which should use native widget for the popup.

[Tony Chang]

Comment on attachment 80987 [details]
Proposed patch.

If it only affects Windows and Linux, this is fine.

For reference, IE & Firefox Win aligns the text in popups.  This is consistent with native menu lists on Windows. GTK+ menu lists also align the text (actually, GTK+ drop downs are like OSX menu lists where the text doesn't move but the options appear above/below the currently selected item).

[WebKit Commit Bot]

Comment on attachment 80987 [details]
Proposed patch.

Clearing flags on attachment: 80987

Committed r77716: <http://trac.webkit.org/changeset/77716>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 80987 [details]:

inspector/elements-panel-xhtml-structure.xhtml bug 53835 (authors: [e-mail] and [e-mail])
The commit-queue is continuing to process your patch.