From 2ba8261fee95cca0d119879a95f871133f0b51d5 Mon Sep 17 00:00:00 2001 From: J-P Nurmi Date: Sat, 27 Jun 2015 00:57:26 +0200 Subject: Control::spacing Change-Id: I67d81db16e95813b1ed9903017ce4d272093f7bb Reviewed-by: J-P Nurmi --- src/imports/calendar/CalendarView.qml | 2 ++ src/imports/calendar/DayOfWeekRow.qml | 1 + src/imports/calendar/WeekNumberColumn.qml | 1 + src/imports/controls/CheckBox.qml | 7 ++++--- src/imports/controls/GroupBox.qml | 3 ++- src/imports/controls/PageIndicator.qml | 3 ++- src/imports/controls/RadioButton.qml | 7 ++++--- src/imports/controls/Switch.qml | 7 ++++--- src/imports/controls/TabBar.qml | 4 +++- src/imports/controls/ToggleButton.qml | 7 ++++--- src/imports/extras/SwipeView.qml | 3 +-- 11 files changed, 28 insertions(+), 17 deletions(-) (limited to 'src/imports') diff --git a/src/imports/calendar/CalendarView.qml b/src/imports/calendar/CalendarView.qml index 2d77c6f4..9a6c8d1e 100644 --- a/src/imports/calendar/CalendarView.qml +++ b/src/imports/calendar/CalendarView.qml @@ -67,6 +67,8 @@ AbstractCalendarView { contentItem: Grid { rows: 6 columns: 7 + rowSpacing: control.spacing + columnSpacing: control.spacing Repeater { model: control.source diff --git a/src/imports/calendar/DayOfWeekRow.qml b/src/imports/calendar/DayOfWeekRow.qml index 50c1cb46..08532527 100644 --- a/src/imports/calendar/DayOfWeekRow.qml +++ b/src/imports/calendar/DayOfWeekRow.qml @@ -63,6 +63,7 @@ AbstractDayOfWeekRow { //! [contentItem] contentItem: Row { + spacing: control.spacing Repeater { model: control.source delegate: control.delegate diff --git a/src/imports/calendar/WeekNumberColumn.qml b/src/imports/calendar/WeekNumberColumn.qml index 7235262b..8c0e75fa 100644 --- a/src/imports/calendar/WeekNumberColumn.qml +++ b/src/imports/calendar/WeekNumberColumn.qml @@ -63,6 +63,7 @@ AbstractWeekNumberColumn { //! [contentItem] contentItem: Column { + spacing: control.spacing Repeater { model: control.source delegate: control.delegate diff --git a/src/imports/controls/CheckBox.qml b/src/imports/controls/CheckBox.qml index 9e71dda9..833a720f 100644 --- a/src/imports/controls/CheckBox.qml +++ b/src/imports/controls/CheckBox.qml @@ -43,7 +43,7 @@ AbstractCheckBox { implicitWidth: Math.max(background ? background.implicitWidth : 0, (label ? label.implicitWidth : 0) + (indicator ? indicator.implicitWidth : 0) + - (label && indicator ? Theme.spacing : 0) + leftPadding + rightPadding) + (label && indicator ? spacing : 0) + leftPadding + rightPadding) implicitHeight: Math.max(background ? background.implicitHeight : 0, Math.max(label ? label.implicitHeight : 0, indicator ? indicator.implicitHeight : 0) + topPadding + bottomPadding) @@ -54,6 +54,7 @@ AbstractCheckBox { Accessible.role: Accessible.CheckBox padding: Theme.padding + spacing: Theme.spacing //! [indicator] indicator: Rectangle { @@ -84,9 +85,9 @@ AbstractCheckBox { //! [label] label: Text { - x: control.mirrored ? control.leftPadding : (indicator.x + indicator.width + control.Theme.spacing) + x: control.mirrored ? control.leftPadding : (indicator.x + indicator.width + control.spacing) y: control.topPadding - width: control.availableWidth - indicator.width - control.Theme.spacing + width: control.availableWidth - indicator.width - control.spacing height: control.availableHeight text: control.text diff --git a/src/imports/controls/GroupBox.qml b/src/imports/controls/GroupBox.qml index cc84008f..7a625a81 100644 --- a/src/imports/controls/GroupBox.qml +++ b/src/imports/controls/GroupBox.qml @@ -46,8 +46,9 @@ AbstractGroupBox { contentWidth: contentChildren.length === 1 ? contentChildren[0].implicitWidth : 0 contentHeight: contentChildren.length === 1 ? contentChildren[0].implicitHeight : 0 + spacing: Theme.spacing padding: Theme.padding - topPadding: Theme.padding + (label && title ? label.implicitHeight + Theme.spacing : 0) + topPadding: Theme.padding + (label && title ? label.implicitHeight + spacing : 0) //! [contentItem] contentItem: Item { } diff --git a/src/imports/controls/PageIndicator.qml b/src/imports/controls/PageIndicator.qml index aa7cbc6f..cb74567d 100644 --- a/src/imports/controls/PageIndicator.qml +++ b/src/imports/controls/PageIndicator.qml @@ -48,6 +48,7 @@ AbstractPageIndicator { contentItem.implicitHeight + topPadding + bottomPadding) padding: Theme.padding + spacing: Theme.spacing //! [delegate] delegate: Rectangle { @@ -64,7 +65,7 @@ AbstractPageIndicator { //! [contentItem] contentItem: Row { - spacing: control.Theme.spacing + spacing: control.spacing Repeater { model: control.count diff --git a/src/imports/controls/RadioButton.qml b/src/imports/controls/RadioButton.qml index 106b382e..cd2b553a 100644 --- a/src/imports/controls/RadioButton.qml +++ b/src/imports/controls/RadioButton.qml @@ -43,7 +43,7 @@ AbstractRadioButton { implicitWidth: Math.max(background ? background.implicitWidth : 0, (label ? label.implicitWidth : 0) + (indicator ? indicator.implicitWidth : 0) + - (label && indicator ? Theme.spacing : 0) + leftPadding + rightPadding) + (label && indicator ? spacing : 0) + leftPadding + rightPadding) implicitHeight: Math.max(background ? background.implicitHeight : 0, Math.max(label ? label.implicitHeight : 0, indicator ? indicator.implicitHeight : 0) + topPadding + bottomPadding) @@ -54,6 +54,7 @@ AbstractRadioButton { Accessible.role: Accessible.RadioButton padding: Theme.padding + spacing: Theme.spacing //! [indicator] indicator: Rectangle { @@ -85,9 +86,9 @@ AbstractRadioButton { //! [label] label: Text { - x: control.mirrored ? control.leftPadding : (indicator.x + indicator.width + control.Theme.spacing) + x: control.mirrored ? control.leftPadding : (indicator.x + indicator.width + control.spacing) y: control.topPadding - width: control.availableWidth - indicator.width - control.Theme.spacing + width: control.availableWidth - indicator.width - control.spacing height: control.availableHeight text: control.text diff --git a/src/imports/controls/Switch.qml b/src/imports/controls/Switch.qml index c1661f35..e61fb152 100644 --- a/src/imports/controls/Switch.qml +++ b/src/imports/controls/Switch.qml @@ -43,7 +43,7 @@ AbstractSwitch { implicitWidth: Math.max(background ? background.implicitWidth : 0, (label ? label.implicitWidth : 0) + (indicator ? indicator.implicitWidth : 0) + - (label && indicator ? Theme.spacing : 0) + leftPadding + rightPadding) + (label && indicator ? spacing : 0) + leftPadding + rightPadding) implicitHeight: Math.max(background ? background.implicitHeight : 0, Math.max(label ? label.implicitHeight : 0, indicator ? indicator.implicitHeight : 0) + topPadding + bottomPadding) @@ -55,6 +55,7 @@ AbstractSwitch { Accessible.role: Accessible.Button // TODO: Switch? padding: Theme.padding + spacing: Theme.spacing //! [indicator] indicator: Rectangle { @@ -94,9 +95,9 @@ AbstractSwitch { //! [label] label: Text { - x: control.mirrored ? control.leftPadding : (indicator.x + indicator.width + control.Theme.spacing) + x: control.mirrored ? control.leftPadding : (indicator.x + indicator.width + control.spacing) y: control.topPadding - width: control.availableWidth - indicator.width - control.Theme.spacing + width: control.availableWidth - indicator.width - control.spacing height: control.availableHeight text: control.text diff --git a/src/imports/controls/TabBar.qml b/src/imports/controls/TabBar.qml index 02363935..229e5f68 100644 --- a/src/imports/controls/TabBar.qml +++ b/src/imports/controls/TabBar.qml @@ -49,11 +49,13 @@ AbstractTabBar { Accessible.role: Accessible.PageTabList + spacing: 1 + //! [contentItem] contentItem: ListView { id: listView - spacing: 1 + spacing: control.spacing orientation: ListView.Horizontal boundsBehavior: Flickable.StopAtBounds snapMode: ListView.SnapToItem diff --git a/src/imports/controls/ToggleButton.qml b/src/imports/controls/ToggleButton.qml index 1655a317..2ecc0f18 100644 --- a/src/imports/controls/ToggleButton.qml +++ b/src/imports/controls/ToggleButton.qml @@ -43,7 +43,7 @@ AbstractToggleButton { implicitWidth: Math.max(background ? background.implicitWidth : 0, (label ? label.implicitWidth : 0) + (indicator ? indicator.implicitWidth : 0) + - (label && indicator ? Theme.spacing : 0) + leftPadding + rightPadding) + (label && indicator ? spacing : 0) + leftPadding + rightPadding) implicitHeight: Math.max(background ? background.implicitHeight : 0, Math.max(label ? label.implicitHeight : 0, indicator ? indicator.implicitHeight : 0) + topPadding + bottomPadding) @@ -55,6 +55,7 @@ AbstractToggleButton { Accessible.role: Accessible.Button padding: Theme.padding + spacing: Theme.spacing //! [indicator] indicator: Rectangle { @@ -93,9 +94,9 @@ AbstractToggleButton { //! [label] label: Text { - x: control.mirrored ? control.leftPadding : (indicator.x + indicator.width + control.Theme.spacing) + x: control.mirrored ? control.leftPadding : (indicator.x + indicator.width + control.spacing) y: control.topPadding - width: control.availableWidth - indicator.width - control.Theme.spacing + width: control.availableWidth - indicator.width - control.spacing height: control.availableHeight text: control.text diff --git a/src/imports/extras/SwipeView.qml b/src/imports/extras/SwipeView.qml index e52a8b44..f234d84a 100644 --- a/src/imports/extras/SwipeView.qml +++ b/src/imports/extras/SwipeView.qml @@ -40,8 +40,6 @@ import QtQuick.Extras 2.0 AbstractSwipeView { id: control - property alias spacing: listView.spacing - implicitWidth: Math.max(background ? background.implicitWidth : 0, contentItem.implicitWidth + leftPadding + rightPadding) implicitHeight: Math.max(background ? background.implicitHeight : 0, @@ -56,6 +54,7 @@ AbstractSwipeView { model: control.contentModel currentIndex: control.currentIndex + spacing: control.spacing orientation: Qt.Horizontal snapMode: ListView.SnapOneItem boundsBehavior: Flickable.StopAtBounds -- cgit v1.2.3