tobast/glowingbear-mainbox
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #896 from lorenzhs/fix-fontsize-unit-fixing

Browse source 
Fix fontsize unit fixing (assume px if no unit specified)
This commit is contained in:
Lorenz Hübschle-Schneider 2017-04-05 14:34:04 +02:00 committed by GitHub
commit 9b2e5d77db
1 changed files with 5 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
js/glowingbear.js
View file

@ -430,6 +430,11 @@ weechat.controller('WeechatCtrl', ['$rootScope', '$scope', '$store', '$timeout',
}); });
// Update font size when changed // Update font size when changed
settings.addCallback('fontsize', function(fontsize) { settings.addCallback('fontsize', function(fontsize) {
if (typeof(fontsize) === "number") {
// settings module recognizes a fontsize without unit it as a number
// and converts, we need to convert back
fontsize = fontsize.toString();
}
// If no unit is specified, it should be pixels // If no unit is specified, it should be pixels
if (fontsize.match(/^[0-9]+$/)) { if (fontsize.match(/^[0-9]+$/)) {
fontsize += 'px'; fontsize += 'px';