Merge lp:~elopio/ubuntu-ui-toolkit/fix1209405-header_object_name into lp:ubuntu-ui-toolkit

Proposed by Leo Arias
Status: Merged
Approved by: Zsombor Egri
Approved revision: 685
Merged at revision: 684
Proposed branch: lp:~elopio/ubuntu-ui-toolkit/fix1209405-header_object_name
Merge into: lp:ubuntu-ui-toolkit
Diff against target: 27 lines (+5/-1)
2 files modified
modules/Ubuntu/Components/MainView.qml (+4/-0)
tests/autopilot/ubuntuuitoolkit/emulators.py (+1/-1)
To merge this branch: bzr merge lp:~elopio/ubuntu-ui-toolkit/fix1209405-header_object_name
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Nicholas Skaggs (community) code Approve
Ubuntu SDK team Pending
Review via email: mp+179060@code.launchpad.net

Commit message

Added the objectName property to the top header.

To post a comment you must log in.
Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

Just a comment; I encountered this issue while using the emulator with the clock core app. The trouble with any generic select_single is the potential for dupes. I understand objectNames are not always desired, but imho, selecting exactly what you want is worth the price and is much more sustainable for us.

Revision history for this message
Nicholas Skaggs (nskaggs) :
review: Approve (code)
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) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'modules/Ubuntu/Components/MainView.qml'
2--- modules/Ubuntu/Components/MainView.qml 2013-07-18 12:27:33 +0000
3+++ modules/Ubuntu/Components/MainView.qml 2013-08-08 19:44:22 +0000
4@@ -267,6 +267,10 @@
5 in \l Page to determine the area for the \l Page to fill.
6 */
7 Header {
8+ // FIXME We need to set an object name to this header in order to differentiate it from the ListItem.Header on Autopilot tests.
9+ // This is a temporary workaround while we find a better solution for https://bugs.launchpad.net/autopilot/+bug/1210265
10+ // --elopio - 2013-08-08
11+ objectName: "MainView_Header"
12 id: headerItem
13 property real bottomY: headerItem.y + headerItem.height
14 }
15
16=== modified file 'tests/autopilot/ubuntuuitoolkit/emulators.py'
17--- tests/autopilot/ubuntuuitoolkit/emulators.py 2013-08-07 17:33:32 +0000
18+++ tests/autopilot/ubuntuuitoolkit/emulators.py 2013-08-08 19:44:22 +0000
19@@ -47,7 +47,7 @@
20
21 def get_header(self):
22 """Return the Header emulator of the MainView."""
23- return self.select_single('Header')
24+ return self.select_single('Header', objectName='MainView_Header')
25
26 def get_toolbar(self):
27 """Return the Toolbar emulator of the MainView."""

Subscribers

People subscribed via source and target branches

to status/vote changes: