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
=== modified file 'config/JsConfig.qml'
--- config/JsConfig.qml 2013-04-30 22:05:00 +0000
+++ config/JsConfig.qml 2013-06-03 16:30:39 +0000
@@ -18,13 +18,13 @@
1818
19 //FontSize19 //FontSize
20 function getFontSize(){20 function getFontSize(){
21 var res = dbEngine.getConfig("fontSize");21 var res = dbEngine.getConfig("fontSize");
22 if( res === "UNKNOWN" ) return "12";22 if( res === "UNKNOWN" || res === "NaN" ) return 14;
23 return res;23 else return parseInt(res);
24 }24 }
2525
26 function setFontSize(size){26 function setFontSize(size){
27 dbEngine.setConfig("fontSize",size);27 dbEngine.setConfig("fontSize",size.toString() );
28 }28 }
2929
30}30}

Subscribers

People subscribed via source and target branches