Bug 80832

Summary: [Qt][WK2] Add support for rudimentary scroll indicators in MiniBrowser
Product: WebKit Reporter: Andras Becsi <abecsi>
Component: Tools / TestsAssignee: Andras Becsi <abecsi>
Status: RESOLVED FIXED    
Severity: Normal CC: hausmann, laszlo.gombos, menard, vestbo, webkit.review.bot, zoltan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: Linux   
Attachments:
Description Flags
proposed patch none

Description Andras Becsi 2012-03-12 06:49:41 PDT
The flickable WebView needs scroll indicators in a similar manner as the ScrollDecorator QML component.
Comment 1 Tor Arne Vestbø 2012-03-12 06:54:19 PDT
I assume this is an intermediate step before we make the WebView inherit Flickable so we can re-use other scroll-indicator-implementations?
Comment 2 Andras Becsi 2012-03-12 07:14:54 PDT
Created attachment 131321 [details]
proposed patch
Comment 3 Andras Becsi 2012-03-12 07:20:54 PDT
(In reply to comment #1)
> I assume this is an intermediate step before we make the WebView inherit Flickable so we can re-use other scroll-indicator-implementations?

Yes this would be an intermediate step, that's why I used the same API as the ScrollDecorator.
Later on we could check whether the platform has a scroll indicator implementation, and if it has none MiniBrowser could use this basic implementation.
Comment 4 Andras Becsi 2012-03-12 11:03:13 PDT
Comment on attachment 131321 [details]
proposed patch

Clearing flags on attachment: 131321

Committed r110447: <http://trac.webkit.org/changeset/110447>
Comment 5 Andras Becsi 2012-03-12 11:03:23 PDT
All reviewed patches have been landed.  Closing bug.