Merge lp:~hiroshidi/ubuntu-terminal-app/font-size-fix into lp:ubuntu-terminal-app

Proposed by Dmitry Zagnoyko
Status: Merged
Approved by: Michael Hall
Approved revision: 10
Merged at revision: 10
Proposed branch: lp:~hiroshidi/ubuntu-terminal-app/font-size-fix
Merge into: lp:ubuntu-terminal-app
Diff against target: 21 lines (+4/-4)
1 file modified
config/JsConfig.qml (+4/-4)
To merge this branch: bzr merge lp:~hiroshidi/ubuntu-terminal-app/font-size-fix
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Michael Hall Approve
Review via email: mp+167087@code.launchpad.net

Commit message

Default font changed to 14 (LP: #1187015)

Description of the change

Quick fix for LP: #1187015

To post a comment you must log in.
Revision history for this message
Michael Hall (mhall119) wrote :

Looks good to me

review: Approve
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'config/JsConfig.qml'
2--- config/JsConfig.qml 2013-04-30 22:05:00 +0000
3+++ config/JsConfig.qml 2013-06-03 16:30:39 +0000
4@@ -18,13 +18,13 @@
5
6 //FontSize
7 function getFontSize(){
8- var res = dbEngine.getConfig("fontSize");
9- if( res === "UNKNOWN" ) return "12";
10- return res;
11+ var res = dbEngine.getConfig("fontSize");
12+ if( res === "UNKNOWN" || res === "NaN" ) return 14;
13+ else return parseInt(res);
14 }
15
16 function setFontSize(size){
17- dbEngine.setConfig("fontSize",size);
18+ dbEngine.setConfig("fontSize",size.toString() );
19 }
20
21 }

Subscribers

People subscribed via source and target branches