kdewebkit moved to kdereview

[Benjamin Meyer]

On Oct 25, 2009, at 5:40 PM, Urs Wolfer wrote:

> On Sunday 25 October 2009 22:05:08 Sebastian Kügler wrote:
>> On Sunday 25 October 2009 20:02:51 Urs Wolfer wrote:
>>> I have just moved the kdewebkit lib from
>>> playground/libs/webkitkde/kdewebkit  into kdereview. It's the KDE
>>> integration part of QtWebKit which is used directly in many apps and
>>> libs already (...which does not include the WebKit KPart). Any KDE  
>>> app is
>>> supposed to move to this integration lib when it is in kdelibs  
>>> (plans are
>>> to move it to kdelibs/kdewebkit).
>>
>> Could you briefly explain what features this brings? That might  
>> make it
>> easier for people to understand why it's needed ...
>
> Sure. The main point is for sure the integration of KIO. Without  
> kdewebkit,
> any app which is using QWeb* classes does network interactions  
> without KDE
> technologies. Other points are for example:
> * using KDE icons in context menu

In 4.6 QtWebKit should be using the QIcon::fromTheme to get the KDE  
icons in the context menu and if it doesn't then we should file a  
patch to QtWebKit.

> * using KDE plugins (for example nsplugins flash)

Can you clarify this?  Starting in Qt 4.5 QtWebKit supported nsplugins  
and kde apps can already use them...

> * KDE dialogs for javascript dialogs (alert, file picker, ..)

The KDE File dialog should already be used starting with I believe KDE  
4.3 when the static QFileDialog pointers were overloaded so anytime  
you call the static QFileDialog functions it will use the KDE File  
Dialog.  As for the alert dialog while it does use QMessageBox,  
QMessageBox should be using the KDE icon from the style which is the  
only thing I could see the KMessageBox adding.

So what goes kdewebkit bring us that we don't either already have or  
will be getting with Qt4.6?

-Benjamin meyer