Ubuntu Settings Components

Merge lp:~nick-dedekind/ubuntu-settings-components/lp1378821.time-translation into lp:~registry/ubuntu-settings-components/trunk

  1. lp1378821.time-translation
  2. Merge into trunk
Proposed by Nick Dedekind on 2015-01-12
Status: Merged
Approved by: Andrea Cimitan on 2015-10-07
Approved revision: 84
Merged at revision: 100
Proposed branch: lp:~nick-dedekind/ubuntu-settings-components/lp1378821.time-translation
Merge into: lp:~registry/ubuntu-settings-components/trunk
Diff against target: 15 lines (+2/-2)
1 file modified
Ubuntu/Settings/Components/MessageHeader.qml (+2/-2)
To merge this branch: bzr merge lp:~nick-dedekind/ubuntu-settings-components/lp1378821.time-translation
Reviewer Review Type Date Requested Status
Andrea Cimitan (community) Approve on 2015-10-07
Lukáš Tinkl (community) 2015-01-12 Approve on 2015-10-07
PS Jenkins bot (community) continuous-integration Approve on 2015-01-12
Review via email: mp+246194@code.launchpad.net

Commit Message

Fixed problem with message time text alignment.

Description of the Change

Fixed problem with message time text alignment.

To post a comment you must log in.
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Lukáš Tinkl (lukas-kde) wrote :

LGTM, approving

review: Approve
Andrea Cimitan (cimi) wrote :

 * Did you perform an exploratory manual test run of the code change and any related functionality?
y
 * Did CI run pass? If not, please explain why.
y

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Ubuntu/Settings/Components/MessageHeader.qml'
2--- Ubuntu/Settings/Components/MessageHeader.qml 2014-09-15 17:18:27 +0000
3+++ Ubuntu/Settings/Components/MessageHeader.qml 2015-01-12 17:28:06 +0000
4@@ -104,10 +104,10 @@
5 Label {
6 id: timeText
7 objectName: "time"
8- elide: Text.ElideRight
9+ anchors.right: parent.right
10+
11 fontSize: "x-small"
12 maximumLineCount: 1
13- horizontalAlignment: Text.AlignRight
14 }
15 spacing: units.gu(0.5)
16

Subscribers

People subscribed via source and target branches

to all changes: