Merge lp://staging/~stan/pasaffe/window_title_and_escaping into lp://staging/~mdeslaur/pasaffe/trunk
Proposed by
Tristan Hill
Status: | Merged |
---|---|
Merge reported by: | Marc Deslauriers |
Merged at revision: | not available |
Proposed branch: | lp://staging/~stan/pasaffe/window_title_and_escaping |
Merge into: | lp://staging/~mdeslaur/pasaffe/trunk |
Diff against target: |
78 lines (+20/-12) 1 file modified
pasaffe/PasaffeWindow.py (+20/-12) |
To merge this branch: | bzr merge lp://staging/~stan/pasaffe/window_title_and_escaping |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Marc Deslauriers | Approve | ||
Review via email: mp+333873@code.staging.launchpad.net |
Description of the change
* set window title based on current loaded file
* escape text e.g. in case of & in entry name
To post a comment you must log in.
Hi! Thanks for the merge proposal.
ACK on the text escaping.
What's the use case on having the filename in the window title though? I feel that is a detail most users don't need to know when they're using the default database location. Could you perhaps make it only display the filename if it isn't the default file location?