Review Request: WebView Kinetic Scrolling

Kenneth Christiansen kenneth.christiansen at openbossa.org
Sat Oct 3 15:40:17 CEST 2009


Would be cool if you guys could have a look at the QGraphicsWebView in
WebKit trunk, so we can get feedback on the API before we release it
with Qt 4.6.

Cheers,
Kenneth

On Sat, Oct 3, 2009 at 10:16 AM, Marco Martin <notmart at gmail.com> wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1764/#review2534
> -----------------------------------------------------------
>
> Ship it!
>
>
> apart a couple of stye issues it's ok
>
>
> trunk/KDE/kdelibs/plasma/widgets/webview.cpp
> <http://reviewboard.kde.org/r/1764/#comment1859>
>
>    if (event->pos().x()
>
>
>
> trunk/KDE/kdelibs/plasma/widgets/webview.cpp
> <http://reviewboard.kde.org/r/1764/#comment1860>
>
>    if (me.isAccepted()) {
>
>
>
> trunk/KDE/kdelibs/plasma/widgets/webview.cpp
> <http://reviewboard.kde.org/r/1764/#comment1861>
>
>    also there, no spaces
>
>
> - Marco
>
>
> On 2009-10-03 13:07:41, igorto wrote:
>>
>> -----------------------------------------------------------
>> This is an automatically generated e-mail. To reply, visit:
>> http://reviewboard.kde.org/r/1764/
>> -----------------------------------------------------------
>>
>> (Updated 2009-10-03 13:07:41)
>>
>>
>> Review request for Plasma, Aaron Seigo, Marco Martin, and Adenilson Cavalcanti.
>>
>>
>> Summary
>> -------
>>
>> Add support for kinetic scrolling in webview widget.
>>
>> TODO:
>>
>> -Make Horizontal Kinetic Scrolling works
>>
>>
>> Diffs
>> -----
>>
>>   trunk/KDE/kdelibs/plasma/widgets/webview.h 1030897
>>   trunk/KDE/kdelibs/plasma/widgets/webview.cpp 1030897
>>
>> Diff: http://reviewboard.kde.org/r/1764/diff
>>
>>
>> Testing
>> -------
>>
>> webbrowser
>>
>>
>> Thanks,
>>
>> igorto
>>
>>
>
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>



-- 
Kenneth Rohde Christiansen
Technical Lead / Software Engineer
Qt Labs Americas, Nokia Technology Institute, INdT
Phone  +55 81 8895 6002 / E-mail kenneth.christiansen at openbossa.org


More information about the Plasma-devel mailing list