Merge lp:~osomon/webbrowser-app/new-panel-api into lp:webbrowser-app

Proposed by Olivier Tilloy
Status: Merged
Merge reported by: Olivier Tilloy
Merged at revision: not available
Proposed branch: lp:~osomon/webbrowser-app/new-panel-api
Merge into: lp:webbrowser-app
Diff against target: 46 lines (+5/-5)
1 file modified
src/app/Browser.qml (+5/-5)
To merge this branch: bzr merge lp:~osomon/webbrowser-app/new-panel-api
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Needs Fixing
Tim Peeters (community) Approve
Review via email: mp+185222@code.launchpad.net

Commit message

Use the new Panel API (open() and close() methods instead of setting the value of the 'opened' property).

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Tim Peeters (tpeeters) wrote :

Code looks good, but it looks as if jenkins uses an UITK version from before the changes that add open() and close() to the Panel.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/app/Browser.qml'
--- src/app/Browser.qml 2013-08-30 15:07:14 +0000
+++ src/app/Browser.qml 2013-09-12 09:14:52 +0000
@@ -170,7 +170,7 @@
170 if (currentWebview) {170 if (currentWebview) {
171 currentWebview.forceActiveFocus()171 currentWebview.forceActiveFocus()
172 }172 }
173 panel.item.opened = false173 panel.item.close()
174 }174 }
175 }175 }
176176
@@ -198,7 +198,7 @@
198 }198 }
199 height: units.gu(8)199 height: units.gu(8)
200200
201 opened: true201 Component.onCompleted: open()
202 onOpenedChanged: {202 onOpenedChanged: {
203 if (!opened) {203 if (!opened) {
204 Qt.inputMethod.hide()204 Qt.inputMethod.hide()
@@ -225,13 +225,13 @@
225 onLoadingChanged: {225 onLoadingChanged: {
226 if (loading) {226 if (loading) {
227 if (panel.item) {227 if (panel.item) {
228 panel.item.opened = true228 panel.item.open()
229 }229 }
230 } else if (stopped) {230 } else if (stopped) {
231 stopped = false231 stopped = false
232 } else if (!addressBar.activeFocus) {232 } else if (!addressBar.activeFocus) {
233 if (panel.item) {233 if (panel.item) {
234 panel.item.opened = false234 panel.item.close()
235 }235 }
236 if (currentWebview) {236 if (currentWebview) {
237 currentWebview.forceActiveFocus()237 currentWebview.forceActiveFocus()
@@ -386,7 +386,7 @@
386 if (!browser.chromeless) {386 if (!browser.chromeless) {
387 if (!url) {387 if (!url) {
388 panel.chrome.addressBar.forceActiveFocus()388 panel.chrome.addressBar.forceActiveFocus()
389 panel.item.opened = true389 panel.item.open()
390 }390 }
391 }391 }
392 }392 }

Subscribers

People subscribed via source and target branches

to status/vote changes: