starttde: Add dynamic determination of default value for forceFontDPIEnable #231
Merged
SlavekB
merged 1 commits from issue/229/starttde-default-forceFontDPIEnable
into master
3 years ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'issue/229/starttde-default-forceFontDPIEnable'
Deleting a branch is permanent. It CANNOT be undone. Continue?
depending on the current value of forceFontDPI. This solves the automatic transition of configuration after adding support for variable font DPI.
It looks good and it's a nice completion on the previous work.
I would rework one line of the script though
# Get current font DPI configuration
CUR_ForceFontDPI="`$TDEDIR/bin/kreadconfig --file kcmfonts --group General --key forceFontDPI --default 0`"
[ $CUR_ForceFontDPI -gt 0 ] && CUR_ForceFontDPIEnable=true || CUR_ForceFontDPIEnable=false
How about changing line 369 into this? Easier to read, takes away any need to think about priority of && and || and order of execution
e3d6d24b51
to48077a30f9
3 years agoLooks good now
48077a30f9
into master 3 years agoReviewers
48077a30f9
.