[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: API Review: Tracking down the last beasties
From:       Kevin Ottens <ervin () kde ! org>
Date:       2008-10-28 23:27:26
Message-ID: 200810290027.36612.ervin () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


Le Tuesday 28 October 2008, Marco Martin a écrit :
> seriously tough, the only problem i perhaps see with webview is that it
> would suggest that as the other widgets is a proxywidget of QWebView, that
> is not since it uses directly QWebPage, so nativeWidget() will never return
> anything meaningful..

That's not a big issue IMO since the target users for our API will probably 
never use nativeWidget() anyway.

My idea is more that QWebView is a view you can use for a QWebPage, 
Plasma::WebView is another.

> perhaps we could call it WebPage or WebPageWidget or something like this

WebPage would be even more confusing IMO, because then you have a setPage() 
method which takes a QWebPage as argument.

WebPageWidget would not be confusing, it just lacks accuracy IMO. "Widget" is 
kind of generic while here it's clearly a view onto something. Also it's kind 
of long for no real extra information.

Regards.
-- 
Kévin 'ervin' Ottens, http://ervin.ipsquad.net
"Ni le maître sans disciple, Ni le disciple sans maître,
Ne font reculer l'ignorance."


["signature.asc" (application/pgp-signature)]

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[prev in list] [next in list] [prev in thread] [next in thread]