Changeset View
Changeset View
Standalone View
Standalone View
kcmkwin/kwineffects/package/contents/ui/Effect.qml
Show First 20 Lines • Show All 58 Lines • ▼ Show 20 Line(s) | 55 | QQC2.CheckBox { | |||
---|---|---|---|---|---|
59 | onToggled: model.StatusRole = checkState | 59 | onToggled: model.StatusRole = checkState | ||
60 | } | 60 | } | ||
61 | 61 | | |||
62 | ColumnLayout { | 62 | ColumnLayout { | ||
63 | Layout.topMargin: Kirigami.Units.smallSpacing | 63 | Layout.topMargin: Kirigami.Units.smallSpacing | ||
64 | Layout.bottomMargin: Kirigami.Units.smallSpacing | 64 | Layout.bottomMargin: Kirigami.Units.smallSpacing | ||
65 | spacing: 0 | 65 | spacing: 0 | ||
66 | 66 | | |||
67 | Kirigami.Heading { | 67 | Kirigami.Heading { | ||
ngraham: Are you sure about this change? Semantically, it is indeed a heading. If the purpose of this… | |||||
68 | Layout.fillWidth: true | 68 | Layout.fillWidth: true | ||
69 | 69 | | |||
70 | level: 4 | 70 | level: 5 | ||
71 | text: model.NameRole | 71 | text: model.NameRole | ||
72 | wrapMode: Text.Wrap | 72 | wrapMode: Text.Wrap | ||
73 | } | 73 | } | ||
74 | 74 | | |||
75 | QQC2.Label { | 75 | QQC2.Label { | ||
76 | Layout.fillWidth: true | 76 | Layout.fillWidth: true | ||
77 | 77 | | |||
78 | text: model.DescriptionRole | 78 | text: model.DescriptionRole | ||
▲ Show 20 Lines • Show All 50 Lines • Show Last 20 Lines |
Are you sure about this change? Semantically, it is indeed a heading. If the purpose of this was to make the list item headings smaller than the section headers by using a Label, you can keep it as a Heading but raise the level to 5, which has the same size as a Label.