Also worth noting, the QtQuick1 WebView was using option 2 (mostly), whether that's a good or a bad thing.
On Wed, 19 Oct 2011 15:27:15 +0200 ext Jocelyn Turcotte <[email protected]> wrote: > Hello list, > > I'm trying to merge as much functionalities between the desktop and touch > view, and the first step is to unify their API regarding page methods and > signals. Right now, QDesktopWebView has all its API in the View class (there > is no Page class) and QTouchWebView has all its API in the Page class (there > are few methods in the View class). > > We either have to: > 1. Create a QDesktopWebPage and move most of the API there (have APIs in the > Page class) > 2. Move most of the API from QTouchWebPage to QTouchWebView (have APIs in the > View class) > > The reason we currently have a Page for touch but not for desktop is to > support anchoring items to the page, like in this example Tor Arne quickly > cooked: https://gist.github.com/1298139 > > === Here is an overview of the pro/cons of putting APIs in the Page class > (1.): > - Resembles what we have for our WebKit1 API. > - Makes the separation cleaner by using the WebView as a container, and put > page related logic under the page. > - Reduces the chances of the Desktop/TouchView becoming a monster class where > everything ends up. > > === Pro/cons for putting APIs in the View class (2.): > - Reduces ambiguity and number of ways to do things: most of QWebView API is > forwarding calls to existing QWebPage or QWebFrame methods, e.g. > QWebView::settings() and QWebPage::settings(). > - Makes using the API a bit simpler by putting all handlers/methods right > under the View, Page and View will always have a 1:1 relation anyway. > > The reason I ask is that the choice isn't obvious and might affect the > flexibility/usability of the API in the future. Do you have a preference or > see other arguments for one or the other? > > best regards and stuff, > Jocelyn > _______________________________________________ > webkit-qt mailing list > [email protected] > http://lists.webkit.org/mailman/listinfo.cgi/webkit-qt _______________________________________________ webkit-qt mailing list [email protected] http://lists.webkit.org/mailman/listinfo.cgi/webkit-qt
