[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