Merge lp:~diegosarmentero/ubuntu-sso-client/window-title into lp:ubuntu-sso-client

Proposed by Diego Sarmentero on 2012-03-09
Status: Merged
Approved by: Diego Sarmentero on 2012-03-09
Approved revision: 906
Merged at revision: 905
Proposed branch: lp:~diegosarmentero/ubuntu-sso-client/window-title
Merge into: lp:ubuntu-sso-client
Diff against target: 48 lines (+8/-1)
3 files modified
ubuntu_sso/qt/__init__.py (+1/-0)
ubuntu_sso/qt/tests/test_ubuntu_sso_wizard.py (+5/-0)
ubuntu_sso/qt/ubuntu_sso_wizard.py (+2/-1)
To merge this branch: bzr merge lp:~diegosarmentero/ubuntu-sso-client/window-title
Reviewer Review Type Date Requested Status
Roberto Alsina (community) Approve on 2012-03-09
Natalia Bidart 2012-03-09 Approve on 2012-03-09
Review via email: mp+96802@code.launchpad.net

Commit Message

- Setting the window title equal to the app_name (LP: #949744).

To post a comment you must log in.
904. By Diego Sarmentero on 2012-03-09

Changing window title to: Ubuntu Single Sign On

905. By Diego Sarmentero on 2012-03-09

changing docstring

Natalia Bidart (nataliabidart) wrote :

Looks good, though I would prefer a constant for the title instead of typing it twice.

review: Approve
906. By Diego Sarmentero on 2012-03-09

Adding a constant for the title name

Roberto Alsina (ralsina) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ubuntu_sso/qt/__init__.py'
2--- ubuntu_sso/qt/__init__.py 2012-03-05 18:56:50 +0000
3+++ ubuntu_sso/qt/__init__.py 2012-03-09 18:39:19 +0000
4@@ -26,6 +26,7 @@
5 ERROR_MESSAGE = 'message'
6 PREFERED_UI_SIZE = {'width': 550, 'height': 525}
7 TITLE_STYLE = u'<span style="font-size:24px">%s</span>'
8+WINDOW_TITLE = 'Ubuntu Single Sign On'
9
10
11 # Based on the gtk implementation
12
13=== modified file 'ubuntu_sso/qt/tests/test_ubuntu_sso_wizard.py'
14--- ubuntu_sso/qt/tests/test_ubuntu_sso_wizard.py 2012-03-05 19:19:41 +0000
15+++ ubuntu_sso/qt/tests/test_ubuntu_sso_wizard.py 2012-03-09 18:39:19 +0000
16@@ -91,6 +91,11 @@
17 finish_button.clicked.emit(True)
18 self.assertEqual(self._called, ((None,), {}))
19
20+ def test_window_title(self):
21+ """Check the window title for the application."""
22+ title = unicode(self.ui.windowTitle())
23+ self.assertEqual(title, ubuntu_sso_wizard.WINDOW_TITLE)
24+
25
26 class UbuntuSSOWizardTestCase(BaseTestCase):
27
28
29=== modified file 'ubuntu_sso/qt/ubuntu_sso_wizard.py'
30--- ubuntu_sso/qt/ubuntu_sso_wizard.py 2012-03-05 21:08:40 +0000
31+++ ubuntu_sso/qt/ubuntu_sso_wizard.py 2012-03-09 18:39:19 +0000
32@@ -30,7 +30,7 @@
33 USER_SUCCESS,
34 )
35 from ubuntu_sso.logger import setup_gui_logging
36-from ubuntu_sso.qt import PREFERED_UI_SIZE
37+from ubuntu_sso.qt import PREFERED_UI_SIZE, WINDOW_TITLE
38 from ubuntu_sso.qt.current_user_sign_in_page import CurrentUserSignInPage
39 from ubuntu_sso.qt.email_verification_page import EmailVerificationPage
40 from ubuntu_sso.qt.error_page import ErrorPage
41@@ -260,6 +260,7 @@
42 logger.debug('UbuntuSSOClientGUI: app_name %r, kwargs %r.',
43 app_name, kwargs)
44 self.app_name = app_name
45+ self.setWindowTitle(WINDOW_TITLE)
46 # create the controller and the ui, then set the cb and call the show
47 # method so that we can work
48 self.wizard = UbuntuSSOWizard(app_name=app_name, **kwargs)

Subscribers

People subscribed via source and target branches