Merge lp:~philip.scott/pantheon-photos/editable-title into lp:~pantheon-photos/pantheon-photos/trunk
Proposed by
Felipe Escoto
Status: | Merged |
---|---|
Approved by: | Danielle Foré |
Approved revision: | 3037 |
Merged at revision: | 3038 |
Proposed branch: | lp:~philip.scott/pantheon-photos/editable-title |
Merge into: | lp:~pantheon-photos/pantheon-photos/trunk |
Diff against target: |
184 lines (+131/-5) 3 files modified
src/CMakeLists.txt (+2/-1) src/sidebar/EditableTitle.vala (+125/-0) src/sidebar/metadata/LibraryProperties.vala (+4/-4) |
To merge this branch: | bzr merge lp:~philip.scott/pantheon-photos/editable-title |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Danielle Foré | Approve | ||
Review via email: mp+310114@code.launchpad.net |
Commit message
LibraryProperties: Change title entry for an EditableTitle
To post a comment you must log in.
I would like to have:
* The whole label be activatable
* save on focus out
I think without those it feels a little awkward