Merge lp:~hiroshidi/ubuntu-terminal-app/plugin-text-display-fix into lp:~ubuntu-terminal-dev/ubuntu-terminal-app/plugin
Proposed by
Dmitry Zagnoyko
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Michael Hall | ||||
Approved revision: | 14 | ||||
Merged at revision: | 13 | ||||
Proposed branch: | lp:~hiroshidi/ubuntu-terminal-app/plugin-text-display-fix | ||||
Merge into: | lp:~ubuntu-terminal-dev/ubuntu-terminal-app/plugin | ||||
Diff against target: |
76 lines (+27/-3) 3 files modified
debian/changelog (+6/-0) konsole-qml-plugin.pro (+3/-0) src/TerminalDisplay.cpp (+18/-3) |
||||
To merge this branch: | bzr merge lp:~hiroshidi/ubuntu-terminal-app/plugin-text-display-fix | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ubuntu Phone Apps Jenkins Bot | continuous-integration | Approve | |
Michael Hall | Approve | ||
Review via email: mp+168365@code.launchpad.net |
Commit message
Fixed text displaying bug (LP: #1182503)
Description of the change
Bug: Command does not appear until enter is pressed.
Cause: float font metric
Fix: force integer font metric
To post a comment you must log in.
PASSED: Continuous integration, rev:13 91.189. 93.125: 8080/job/ ubuntu- terminal- app-plugin- ci/3/ 91.189. 93.125: 8080/job/ ubuntu- terminal- app-plugin- quantal- amd64-ci/ 1 91.189. 93.125: 8080/job/ ubuntu- terminal- app-plugin- raring- amd64-ci/ 3
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild: 91.189. 93.125: 8080/job/ ubuntu- terminal- app-plugin- ci/3/rebuild
http://