Differential D11064 Diff 28792 kcms/fonts/package/contents/ui/main.qml

Changeset View

Standalone View

View Options

kcms/fonts/package/contents/ui/main.qml

Show All 13 Lines1/*
14 You should have received a copy of the GNU Library General Public License 14​ You should have received a copy of the GNU Library General Public License
15 along with this library; see the file COPYING.LIB. If not, write to 15​ along with this library; see the file COPYING.LIB. If not, write to
16 the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, 16​ the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
17 Boston, MA 02110-1301, USA. 17​ Boston, MA 02110-1301, USA.
18*/ 18​*/
19 19​
20import QtQuick 2.1 20​import QtQuick 2.1
21import QtQuick.Layouts 1.1 21​import QtQuick.Layouts 1.1
22import QtQuick.Controls 2.0 as QtControls 22​import QtQuick.Controls 2.0 as QtControls
broulikUnsubmitted
Done

Is 2.2 also sufficient (Qt 5.9)?

broulik: Is `2.2` also sufficient (Qt 5.9)?
progwolffAuthorUnsubmitted
Done

Seems like even 2.0 is okay.
I misinterpreted the docs where it says import QtQuick.Controls 2.3.

progwolff: Seems like even `2.0` is okay. I misinterpreted the docs where it says `import QtQuick.Controls…
23import QtQuick.Dialogs 1.2 as QtDialogs 23​import QtQuick.Dialogs 1.2 as QtDialogs
24import org.kde.kirigami 2.3 as Kirigami 24​import org.kde.kirigami 2.3 as Kirigami
25import org.kde.kcm 1.1 as KCM 25​import org.kde.kcm 1.1 as KCM
26 26​
27KCM.SimpleKCM { 27​KCM.SimpleKCM {
28 id: root 28​ id: root
29 29​
30 KCM.ConfigModule.quickHelp: i18n("Fonts") 30​ KCM.ConfigModule.quickHelp: i18n("Fonts")
▲ Show 20 Lines • Show All 102 Lines • ▼ Show 20 Line(s)132 QtControls.ComboBox {
133 id: subPixelCombo 133​ id: subPixelCombo
134 Layout.preferredWidth: formLayout.maxImplicitWidth 134​ Layout.preferredWidth: formLayout.maxImplicitWidth
135 Kirigami.FormData.label: i18n("Sub-pixel rendering type:") 135​ Kirigami.FormData.label: i18n("Sub-pixel rendering type:")
136 currentIndex: kcm.fontAASettings.subPixelCurrentIndex 136​ currentIndex: kcm.fontAASettings.subPixelCurrentIndex
137 onCurrentIndexChanged: kcm.fontAASettings.subPixelCurrentIndex = currentIndex; 137​ onCurrentIndexChanged: kcm.fontAASettings.subPixelCurrentIndex = currentIndex;
138 model: kcm.fontAASettings.subPixelOptionsModel 138​ model: kcm.fontAASettings.subPixelOptionsModel
139 textRole: "display" 139​ textRole: "display"
140 enabled: antiAliasingComboBox.currentIndex == 0 140​ enabled: antiAliasingComboBox.currentIndex == 0
141​ delegate: QtControls.ItemDelegate {
142​ id: subPixelDelegate
143​ width: subPixelComboImage.implicitWidth
broulikUnsubmitted
Done

The popup width is too small, the text cut off. Either make it larger or at least elide the text on the right

broulik: The popup width is too small, the text cut off. Either make it larger or at least elide the…
144​ contentItem: ColumnLayout {
145​ id: subPixelLayout
146​ width: subPixelComboImage.implicitWidth
147​ QtControls.Label {
broulikUnsubmitted
Done

Use QtQuick Controls Label instead of plain Text for proper rendering and text color

broulik: Use QtQuick Controls `Label` instead of plain `Text` for proper rendering and text color
148​ id: subPixelComboText
149​ text: model.display
150​ }
151​ Image {
152​ id: subPixelComboImage
153​ source: "image://preview/" + model.index + "_" + kcm.fontAASettings.hintingCurrentIndex + ".png"
broulikUnsubmitted
Done

This is the default, no need to explicitly specify

broulik: This is the default, no need to explicitly specify
154​ // Setting sourceSize here is necessary as a workaround for QTBUG-38127
broulikUnsubmitted
Done

Also not needed

broulik: Also not needed
155​ sourceSize.width: 1
broulikUnsubmitted
Done

Coding style, add spaces between:

source: "image://preview/" + model.index + "_" + kcm.fontAASettings.hintCurrentIndex + ".png"
broulik: Coding style, add spaces between: ``` source: "image://preview/" + model.index + "_" + kcm.
156​ sourceSize.height: 1
157​ }
158​ }
159​ }
141 } 160​ }
142 161​
143 QtControls.ComboBox { 162​ QtControls.ComboBox {
144 id: hintingCombo 163​ id: hintingCombo
145 Layout.preferredWidth: formLayout.maxImplicitWidth 164​ Layout.preferredWidth: formLayout.maxImplicitWidth
146 Kirigami.FormData.label: i18n("Hinting style:") 165​ Kirigami.FormData.label: i18n("Hinting style:")
147 currentIndex: kcm.fontAASettings.hintingCurrentIndex 166​ currentIndex: kcm.fontAASettings.hintingCurrentIndex
148 onCurrentTextChanged: kcm.fontAASettings.hintingCurrentIndex = currentIndex; 167​ onCurrentTextChanged: kcm.fontAASettings.hintingCurrentIndex = currentIndex;
149 model: kcm.fontAASettings.hintingOptionsModel 168​ model: kcm.fontAASettings.hintingOptionsModel
150 textRole: "display" 169​ textRole: "display"
151 enabled: antiAliasingComboBox.currentIndex == 0 170​ enabled: antiAliasingComboBox.currentIndex == 0
171​ delegate: QtControls.ItemDelegate {
172​ id: hintingDelegate
173​ width: hintingComboImage.implicitWidth
174​ contentItem: ColumnLayout {
175​ id: hintingLayout
176​ width: hintingComboImage.implicitWidth
177​ QtControls.Label {
178​ id: hintingComboText
179​ text: model.display
180​ }
181​ Image {
182​ id: hintingComboImage
183​ source: "image://preview/" + kcm.fontAASettings.subPixelCurrentIndex + "_" + model.index + ".png"
184​ // Setting sourceSize here is necessary as a workaround for QTBUG-38127
185​ sourceSize.width: 1
davidedmundsonUnsubmitted
Done

We dont' need these transforms.
There shouldn't be any high DPI specicfic code in QML.

davidedmundson: We dont' need these transforms. There shouldn't be any high DPI specicfic code in QML.
186​ sourceSize.height: 1
187​ }
188​ }
189​ }
152 } 190​ }
153 191​
154 RowLayout { 192​ RowLayout {
155 QtControls.CheckBox { 193​ QtControls.CheckBox {
156 id: dpiCheckBox 194​ id: dpiCheckBox
157 checked: dpiSpinBox.value != 96 195​ checked: dpiSpinBox.value != 96
158 text: i18n("Force Fonts DPI:") 196​ text: i18n("Force Fonts DPI:")
159 onCheckedChanged: { 197​ onCheckedChanged: {
Show All 36 Lines