Merge lp:~abreu-alexandre/webbrowser-app/remove-open-in-browser-button-in-popups into lp:webbrowser-app
Proposed by
Alexandre Abreu
Status: | Approved |
---|---|
Approved by: | Alberto Mardegan |
Approved revision: | 1503 |
Proposed branch: | lp:~abreu-alexandre/webbrowser-app/remove-open-in-browser-button-in-popups |
Merge into: | lp:webbrowser-app |
Diff against target: |
81 lines (+26/-3) 3 files modified
src/app/webcontainer/PopupWindowController.qml (+9/-2) src/app/webcontainer/PopupWindowOverlay.qml (+2/-1) tests/autopilot/webapp_container/tests/test_popup_webview_overlay.py (+15/-0) |
To merge this branch: | bzr merge lp:~abreu-alexandre/webbrowser-app/remove-open-in-browser-button-in-popups |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Alberto Mardegan (community) | Approve | ||
system-apps-ci-bot | continuous-integration | Needs Fixing | |
Review via email: mp+300633@code.launchpad.net |
Commit message
Remove the "open in the webrowser" button in an Overlay created as Popups
Description of the change
Remove the "open in the webrowser" button in an Overlay created as Popups
To post a comment you must log in.
Unmerged revisions
- 1503. By Alexandre Abreu
-
Improve overlay open in browser button visibility predicate's readability
- 1502. By Alexandre Abreu
-
add tests
- 1501. By Alexandre Abreu
-
draft
FAILED: Continuous integration, rev:1502 /jenkins. canonical. com/system- apps/job/ lp-webbrowser- app-ci/ 586/ /jenkins. canonical. com/system- apps/job/ build/1010/ console /jenkins. canonical. com/system- apps/job/ build-0- fetch/1010 /jenkins. canonical. com/system- apps/job/ build-1- sourcepkg/ release= vivid+overlay/ 908 /jenkins. canonical. com/system- apps/job/ build-1- sourcepkg/ release= xenial+ overlay/ 908 /jenkins. canonical. com/system- apps/job/ build-1- sourcepkg/ release= yakkety/ 908 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=amd64, release= vivid+overlay/ 902/console /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=amd64, release= xenial+ overlay/ 902/console /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=amd64, release= yakkety/ 902 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=amd64, release= yakkety/ 902/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=armhf, release= vivid+overlay/ 902 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=armhf, release= vivid+overlay/ 902/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=armhf, release= xenial+ overlay/ 902 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=armhf, release= xenial+ overlay/ 902/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=armhf, release= yakkety/ 902 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=armhf, release= yakkety/ 902/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=i386, release= vivid+overlay/ 902/console /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=i386, release= xenial+ overlay/ 902 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=i386, release= xenial+ overlay/ 902/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=i386, release= yakkety/ 902 /jenkins. canonical. com/system- apps/job/ build-2- binpkg/ arch=i386, release= yakkety/ 902/artifact/ output/ *zip*/output. zip
https:/
Executed test runs:
FAILURE: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
FAILURE: https:/
FAILURE: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
FAILURE: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild: /jenkins. canonical. com/system- apps/job/ lp-webbrowser- app-ci/ 586/rebuild
https:/