Changeset View
Changeset View
Standalone View
Standalone View
applets/batterymonitor/package/contents/ui/PopupDialog.qml
Show All 35 Lines | 27 | FocusScope { | |||
---|---|---|---|---|---|
36 | property bool isBrightnessAvailable | 36 | property bool isBrightnessAvailable | ||
37 | property bool isKeyboardBrightnessAvailable | 37 | property bool isKeyboardBrightnessAvailable | ||
38 | 38 | | |||
39 | signal powermanagementChanged(bool checked) | 39 | signal powermanagementChanged(bool checked) | ||
40 | 40 | | |||
41 | Column { | 41 | Column { | ||
42 | id: settingsColumn | 42 | id: settingsColumn | ||
43 | anchors.horizontalCenter: parent.horizontalCenter | 43 | anchors.horizontalCenter: parent.horizontalCenter | ||
44 | width: parent.width - units.gridUnit | 44 | width: parent.width - units.smallSpacing * 2 | ||
45 | spacing: Math.round(units.gridUnit / 2) | 45 | spacing: units.gridUnit | ||
46 | 46 | | |||
47 | Components.Label { | 47 | Components.Label { | ||
48 | // this is just for metrics, TODO use TextMetrics in 5.4 instead | 48 | // this is just for metrics, TODO use TextMetrics in 5.4 instead | ||
49 | id: percentageMeasurementLabel | 49 | id: percentageMeasurementLabel | ||
50 | text: i18nc("Used for measurement", "100%") | 50 | text: i18nc("Used for measurement", "100%") | ||
51 | visible: false | 51 | visible: false | ||
52 | } | 52 | } | ||
53 | 53 | | |||
▲ Show 20 Lines • Show All 48 Lines • ▼ Show 20 Line(s) | 84 | BrightnessItem { | |||
102 | } | 102 | } | ||
103 | } | 103 | } | ||
104 | } | 104 | } | ||
105 | 105 | | |||
106 | PlasmaExtras.ScrollArea { | 106 | PlasmaExtras.ScrollArea { | ||
107 | anchors { | 107 | anchors { | ||
108 | horizontalCenter: parent.horizontalCenter | 108 | horizontalCenter: parent.horizontalCenter | ||
109 | top: settingsColumn.bottom | 109 | top: settingsColumn.bottom | ||
110 | topMargin: units.gridUnit // not divided by 2 for unified looks | 110 | topMargin: units.gridUnit | ||
111 | leftMargin: units.smallSpacing | ||||
111 | bottom: dialog.bottom | 112 | bottom: dialog.bottom | ||
112 | } | 113 | } | ||
113 | width: parent.width - units.gridUnit | 114 | width: parent.width - units.smallSpacing * 2 | ||
114 | 115 | | |||
115 | ListView { | 116 | ListView { | ||
116 | id: batteryList | 117 | id: batteryList | ||
117 | 118 | | |||
118 | boundsBehavior: Flickable.StopAtBounds | 119 | boundsBehavior: Flickable.StopAtBounds | ||
119 | spacing: Math.round(units.gridUnit / 2) | 120 | spacing: Math.round(units.smallSpacing / 2) | ||
gvgeo: I don't see why decrease this spacing.
Were you able to verify this? I don't have access to… | |||||
120 | 121 | | |||
121 | KeyNavigation.tab: brightnessSlider | 122 | KeyNavigation.tab: brightnessSlider | ||
122 | KeyNavigation.backtab: pmSwitch | 123 | KeyNavigation.backtab: pmSwitch | ||
123 | 124 | | |||
124 | delegate: BatteryItem { | 125 | delegate: BatteryItem { | ||
125 | width: parent.width | 126 | width: parent.width | ||
126 | battery: model | 127 | battery: model | ||
127 | } | 128 | } | ||
128 | } | 129 | } | ||
129 | } | 130 | } | ||
130 | 131 | | |||
131 | } | 132 | } |
I don't see why decrease this spacing.
Were you able to verify this? I don't have access to two batteries, but I expect this to look bad.