Merge lp:~seb128/unity8/set-inline-reply-hint into lp:unity8
| Status: | Merged |
|---|---|
| Approved by: | kevin gunn on 2015-01-23 |
| Approved revision: | 1436 |
| Merged at revision: | 1570 |
| Proposed branch: | lp:~seb128/unity8/set-inline-reply-hint |
| Merge into: | lp:unity8 |
| Diff against target: |
255 lines (+45/-40) 3 files modified
debian/control (+1/-1) po/unity8.pot (+43/-39) qml/Panel/Indicators/MessageMenuItemFactory.qml (+1/-0) |
| To merge this branch: | bzr merge lp:~seb128/unity8/set-inline-reply-hint |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| PS Jenkins bot | continuous-integration | Needs Fixing on 2015-01-23 | |
| Nick Dedekind (community) | 2015-01-22 | Approve on 2015-01-22 | |
| Michał Sawicz | 2014-11-05 | Needs Fixing on 2014-11-11 | |
|
Review via email:
|
|||
Commit Message
Set text hint property for the messaging menus inline reply widget
Description of the Change
Set text hint property for the messaging menus inline reply widget
NOTE: needs to go lockstep with lp:~seb128/ubuntu-settings-components/define-text-hint-property
| Sebastien Bacher (seb128) wrote : | # |
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1420
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Sebastien Bacher (seb128) wrote : | # |
Done (rebased on the current vcs version as well)
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1421
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1433
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
Text conflict in debian/control
1 conflicts encountered.
| Sebastien Bacher (seb128) wrote : | # |
would the change be ok otherwise? I can rebase if that's to get the change in, no point if that's to have it sitting there again unreviewed until it conflicts because some other changes landed while it was waiting
| Sebastien Bacher (seb128) wrote : | # |
ok, rebased on trunk
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1434
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Nick Dedekind (nick-dedekind) wrote : | # |
Fix is good, but..
./po/update-
| Sebastien Bacher (seb128) wrote : | # |
Rebased on trunk and updated the translation template
| kevin gunn (kgunn72) wrote : | # |
NOTE: needs to go lockstep with lp:~seb128/ubuntu-settings-components/define-text-hint-property
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1436
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://

Requires https:/ /code.launchpad .net/~seb128/ ubuntu- settings- components/ define- text-hint- property/ +merge/ 240706
Doing it this way should avoid having to do translations from the components