[Merge] lp:~fboucault/webbrowser-app/startup_time into lp:webbrowser-app
Florian Boucault
florian.boucault at canonical.com
Fri Aug 26 11:59:27 UTC 2016
Diff comments:
>
> === modified file 'src/app/webbrowser/Browser.qml'
> --- src/app/webbrowser/Browser.qml 2016-08-26 09:18:47 +0000
> +++ src/app/webbrowser/Browser.qml 2016-08-26 09:18:47 +0000
> @@ -222,12 +217,16 @@
> fill: tabContainer
> topMargin: (chrome.state == "shown") ? chrome.height : 0
> }
> - sourceComponent: ErrorSheet {
> - visible: currentWebview ? currentWebview.lastLoadFailed : false
> - url: currentWebview ? currentWebview.url : ""
> + Component.onCompleted: setSource("../ErrorSheet.qml", {
> + "visible": Qt.binding(function(){ return currentWebview ? currentWebview.lastLoadFailed : false }),
> + "url": Qt.binding(function(){ return currentWebview ? currentWebview.url : "" })
> + })
> + Connections {
> + target: errorSheetLoader.item ? errorSheetLoader.item : null
> onRefreshClicked: currentWebview.reload()
> }
> - focus: item.visible
> +
> + focus: item ? item.visible : false
Fixed in rev 1509
> asynchronous: true
> }
>
> @@ -250,7 +252,7 @@
> currentWebview.resetCertificateError()
> }
> }
> - focus: item.visible
> + focus: item ? item.visible : false
Fixed in rev 1509
> asynchronous: true
> }
>
--
https://code.launchpad.net/~fboucault/webbrowser-app/startup_time/+merge/300730
Your team Ubuntu Phablet Team is subscribed to branch lp:webbrowser-app.
More information about the Ubuntu-reviews
mailing list