The use case is applications where the user is usually interested in
monospaced/fixe-width fonts, e.g. using Kate to edit source code.
Remove one redundant ++row.
Depends on D28271
dfaure | |
cfeck | |
apol | |
bport |
Frameworks |
The use case is applications where the user is usually interested in
monospaced/fixe-width fonts, e.g. using Kate to edit source code.
Remove one redundant ++row.
Depends on D28271
See kfontchooserdialogtest app
No Linters Available |
No Unit Test Coverage |
Buildable 24204 | |
Build 24222: arc lint + arc unit |
src/kfontchooser.cpp | ||
---|---|---|
347 | Yes, that's after setting the font family/style/size list views and the size DoubleSpinBox. |
Handle the case where KFontChooser::FixedFontsOnly is set, the fixedOnlyCheckBox should be shown as "checked" from the get go.
src/kfontchooser.cpp | ||
---|---|---|
392 | would true be more readable? (same for the previous line, actually) |
src/kfontchooser.cpp | ||
---|---|---|
392 | Indeed; however onlyFixedCheckbox->setChecked(usingFixed) doesn't need to be in that if block at all. |