[Webkit-unassigned] [Bug 65896] [Qt] Need spin-button implementation

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=65896





--- Comment #12 from Jarred Nicholls <jarred at sencha.com>  2011-08-26 06:19:48 PST ---
(In reply to comment #11)
> (From update of attachment 105306 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=105306&action=review
> 
> >>> Source/WebCore/platform/qt/RenderThemeQt.cpp:1075
> >>> +    int width = ScrollbarTheme::nativeTheme()->scrollbarThickness();
> >> 
> >> So this will mean that on mobile platforms such as the N9 where we are not showing scrollbars, the width will be 0. Not good.
> > 
> > Yeah I was aware of that, and was counting on someone from Nokia to clarify what value (if any) should be given here for QT_MOBILE_THEME, and if we need to add to QtMobileWebTheme's drawComplexControl.  Or, do we keep it the status quo.
> 
> I think mobile platforms need a way other than inner-spin-button because it would be hard to click it.
> Mobile Safari on iOS and Android Browser don't have inner-spin-button.

Precisely.  Andreas Kling thinks it should be something that's delegated to the platformplugin.  I'll wait to hear for next steps, if any, and then will submit a revised patch based on Kenneth's comments.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.