Changeset View
Changeset View
Standalone View
Standalone View
applets/digital-clock/package/contents/ui/configTimeZones.qml
| Show All 12 Lines | |||||
| 13 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | 13 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
| 14 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | 14 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
| 15 | * GNU General Public License for more details. | 15 | * GNU General Public License for more details. | ||
| 16 | * | 16 | * | ||
| 17 | * You should have received a copy of the GNU General Public License | 17 | * You should have received a copy of the GNU General Public License | ||
| 18 | * along with this program. If not, see <http://www.gnu.org/licenses/> | 18 | * along with this program. If not, see <http://www.gnu.org/licenses/> | ||
| 19 | */ | 19 | */ | ||
| 20 | 20 | | |||
| 21 | import QtQuick 2.0 | 21 | import QtQuick 2.13 | ||
| 22 | import QtQuick.Controls 1.2 as QtControls | 22 | import QtQuick.Controls 2.13 as QQC2 | ||
ngraham: `import QtQuick.Controls 2.5 as QQC2` | |||||
| 23 | import QtQuick.Layouts 1.0 | 23 | import QtQuick.Layouts 1.0 | ||
Next plasma release can only depend on Qt 5.12 Please can you change accordingly davidedmundson: Next plasma release can only depend on Qt 5.12
Please can you change accordingly | |||||
| 24 | import QtQuick.Dialogs 1.1 | 24 | import QtQuick.Dialogs 1.1 | ||
| 25 | 25 | | |||
| 26 | import org.kde.plasma.private.digitalclock 1.0 | 26 | import org.kde.plasma.private.digitalclock 1.0 | ||
| 27 | import org.kde.plasma.core 2.0 as PlasmaCore | 27 | import org.kde.plasma.core 2.0 as PlasmaCore | ||
| 28 | import org.kde.kirigami 2.5 as Kirigami | 28 | import org.kde.kirigami 2.5 as Kirigami | ||
| 29 | 29 | | |||
| 30 | ColumnLayout { | 30 | ColumnLayout { | ||
| 31 | id: timeZonesPage | 31 | id: timeZonesPage | ||
| 32 | 32 | | |||
| 33 | property alias cfg_selectedTimeZones: timeZones.selectedTimeZones | 33 | property alias cfg_selectedTimeZones: timeZones.selectedTimeZones | ||
| 34 | property alias cfg_wheelChangesTimezone: enableWheelCheckBox.checked | 34 | property alias cfg_wheelChangesTimezone: enableWheelCheckBox.checked | ||
| 35 | 35 | | |||
| 36 | ColumnLayout { | ||||
| 37 | anchors.fill: parent | ||||
| 38 | | ||||
| 36 | TimeZoneModel { | 39 | TimeZoneModel { | ||
| 37 | id: timeZones | 40 | id: timeZones | ||
| 38 | 41 | | |||
| 39 | onSelectedTimeZonesChanged: { | 42 | onSelectedTimeZonesChanged: { | ||
| 40 | if (selectedTimeZones.length === 0) { | 43 | if (selectedTimeZones.length === 0) { | ||
| 41 | messageWidget.visible = true; | 44 | messageWidget.visible = true; | ||
| 42 | 45 | | |||
| 43 | timeZones.selectLocalTimeZone(); | 46 | timeZones.selectLocalTimeZone(); | ||
| 44 | } | 47 | } | ||
| 45 | } | 48 | } | ||
| 46 | } | 49 | } | ||
| 47 | 50 | | |||
While you're doing some porting, this entire thing can be replaced with a Kirigami.InlineMessage which will drastically reduce the amount of code here, and you can also remove the PlasmaComponents import (PlasmaCore is still needed for units, unless you want to also port things to use Kirigami.Units instead) ngraham: While you're doing some porting, this entire thing can be replaced with a `Kirigami. | |||||
You can take my changes from D24853 kmaterka: You can take my changes from D24853 | |||||
| 48 | // This is just for getting the column width | | |||
| 49 | QtControls.CheckBox { | | |||
| 50 | id: checkbox | | |||
| 51 | visible: false | | |||
| 52 | } | | |||
| 53 | | ||||
| 54 | Kirigami.InlineMessage { | 51 | Kirigami.InlineMessage { | ||
| 55 | id: messageWidget | 52 | id: messageWidget | ||
| 56 | | ||||
| 57 | Layout.fillWidth: true | 53 | Layout.fillWidth: true | ||
| 58 | 54 | Layout.margins: Kirigami.Units.smallSpacing | |||
| 59 | type: Kirigami.MessageType.Warning | 55 | type: Kirigami.MessageType.Warning | ||
| 60 | text: i18n("At least one time zone needs to be enabled. 'Local' was enabled automatically.") | 56 | text: i18n("At least one time zone needs to be enabled. 'Local' was enabled automatically.") | ||
| 61 | | ||||
| 62 | showCloseButton: true | 57 | showCloseButton: true | ||
| 63 | } | 58 | } | ||
| 64 | 59 | | |||
| 65 | QtControls.TextField { | 60 | QQC2.TextField { | ||
| 66 | id: filter | 61 | id: filter | ||
| 67 | Layout.fillWidth: true | 62 | Layout.fillWidth: true | ||
| 68 | placeholderText: i18n("Search Time Zones") | 63 | placeholderText: i18n("Search Time Zones") | ||
| 69 | } | 64 | } | ||
| 70 | 65 | | |||
| 71 | QtControls.TableView { | 66 | QQC2.ScrollView { | ||
| 72 | id: timeZoneView | 67 | id: scroll | ||
| 73 | | ||||
| 74 | signal toggleCurrent | | |||
| 75 | | ||||
| 76 | Layout.fillWidth: true | 68 | Layout.fillWidth: true | ||
| 77 | Layout.fillHeight: true | 69 | Layout.fillHeight: true | ||
ngraham: `units.largeSpacing` is preferable to hardcoded spacing values | |||||
| 78 | 70 | | |||
| 79 | Keys.onSpacePressed: toggleCurrent() | 71 | ListView { | ||
| 72 | id: timeZoneView | ||||
| 73 | focus : true | ||||
| 74 | keyNavigationEnabled: true | ||||
| 80 | 75 | | |||
| 81 | model: TimeZoneFilterProxy { | 76 | model: TimeZoneFilterProxy { | ||
| 82 | sourceModel: timeZones | 77 | sourceModel: timeZones | ||
| 83 | filterString: filter.text | 78 | filterString: filter.text | ||
| 84 | } | 79 | } | ||
| 85 | 80 | | |||
| 86 | QtControls.TableViewColumn { | 81 | delegate: QQC2.CheckDelegate { | ||
| 87 | role: "checked" | 82 | id: checkbox | ||
| 88 | width: checkbox.width | 83 | width: parent.width | ||
| 89 | delegate: | 84 | text: !city || city.indexOf("UTC") === 0 ? comment : comment ? i18n("%1, %2 (%3)", city, region, comment) : i18n("%1, %2", city, region) | ||
| 90 | QtControls.CheckBox { | 85 | checked: model.checked | ||
| 91 | id: checkBox | 86 | onToggled: { | ||
| 92 | anchors.centerIn: parent | 87 | model.checked = checkbox.checked | ||
| 93 | checked: styleData.value | | |||
| 94 | activeFocusOnTab: false // only let the TableView as a whole get focus | | |||
| 95 | onClicked: { | | |||
| 96 | //needed for model's setData to be called | | |||
| 97 | model.checked = checked; | | |||
| 98 | } | | |||
| 99 | | ||||
| 100 | Connections { | | |||
| 101 | target: timeZoneView | | |||
| 102 | onToggleCurrent: { | | |||
| 103 | if (styleData.row === timeZoneView.currentRow) { | | |||
| 104 | model.checked = !checkBox.checked | | |||
| 105 | } | | |||
| 106 | } | | |||
| 107 | } | 88 | } | ||
| 108 | } | 89 | } | ||
| 109 | 90 | | |||
| 110 | resizable: false | 91 | highlight: Rectangle { color: "lightsteelblue"; radius: 5 } | ||
This is pretty hard to follow. I would recommend expanding it to a conventional if/else block for clarity. ngraham: This is pretty hard to follow. I would recommend expanding it to a conventional if/else block… | |||||
| 111 | movable: false | 92 | | ||
| 112 | } | | |||
| 113 | QtControls.TableViewColumn { | | |||
| 114 | role: "city" | | |||
| 115 | title: i18n("City") | | |||
| 116 | } | | |||
| 117 | QtControls.TableViewColumn { | | |||
| 118 | role: "region" | | |||
| 119 | title: i18n("Region") | | |||
| 120 | } | | |||
| 121 | QtControls.TableViewColumn { | | |||
| 122 | role: "comment" | | |||
| 123 | title: i18n("Comment") | | |||
| 124 | } | 93 | } | ||
| 125 | } | 94 | } | ||
| 126 | 95 | | |||
| 127 | RowLayout { | 96 | RowLayout { | ||
| 128 | Layout.fillWidth: true | 97 | Layout.fillWidth: true | ||
| 129 | QtControls.CheckBox { | 98 | QQC2.CheckBox { | ||
| 130 | id: enableWheelCheckBox | 99 | id: enableWheelCheckBox | ||
| 131 | text: i18n("Switch time zone with mouse wheel") | 100 | text: i18n("Switch time zone with mouse wheel") | ||
| 132 | } | 101 | } | ||
| 133 | } | 102 | } | ||
| 134 | } | 103 | } | ||
| 104 | } | ||||
import QtQuick.Controls 2.5 as QQC2