Summary: | [WPE] Add support for rendering spin buttons | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||
Component: | WPE WebKit | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | aperez, bugs-noreply | ||||
Priority: | P2 | ||||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | 207471 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Carlos Garcia Campos
2020-02-13 03:10:52 PST
Created attachment 390626 [details]
Patch
Comment on attachment 390626 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=390626&action=review r+, but please check the comment below before landing. > Source/WebCore/platform/wpe/RenderThemeWPE.cpp:50 > +static const unsigned menuListButtonArrowSize = 16; Does this need to have the same value as “arrowSize“ in ThemeWPE.cpp? If yes, please add a comment here saying so. > Source/WebCore/platform/wpe/ThemeWPE.cpp:40 > +static const unsigned arrowSize = 16; If this needs to have the same size as “menuListButtonArrowSize” in RenderThemeWPE.cpp, please add a comment indicating that. (In reply to Adrian Perez from comment #2) > Comment on attachment 390626 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=390626&action=review > > r+, but please check the comment below before landing. > > > Source/WebCore/platform/wpe/RenderThemeWPE.cpp:50 > > +static const unsigned menuListButtonArrowSize = 16; > > Does this need to have the same value as “arrowSize“ in ThemeWPE.cpp? > If yes, please add a comment here saying so. Not necessarily, the arrow can be scaled, but we never scale it in case of menu list button. It's true that we don't scale it because we know the arrow size is 16 too, but still. > > Source/WebCore/platform/wpe/ThemeWPE.cpp:40 > > +static const unsigned arrowSize = 16; > > If this needs to have the same size as “menuListButtonArrowSize” in > RenderThemeWPE.cpp, please add a comment indicating that. Committed r256823: <https://trac.webkit.org/changeset/256823> |