File 2000-toolbars.patch of Package spectacle

Overview Repositories Revisions Requests Users Attributes Meta

File 2000-toolbars.patch of Package spectacle (Revision 180d56dc52616152104744a2f77adea2)

Currently displaying revision 180d56dc52616152104744a2f77adea2 , Show latest

diff --git a/src/Gui/ButtonGrid.qml b/src/Gui/ButtonGrid.qml
index e14c30e7e8c4ff8bd6f00eba248c88cf8ae4f333..e9d1388b779b916c2ff4ed0e0fab6421f14f31f5 100644
--- a/src/Gui/ButtonGrid.qml
+++ b/src/Gui/ButtonGrid.qml
@@ -10,7 +10,7 @@ import org.kde.spectacle.private 1.0
 
 Grid {
     id: root
-    property int displayMode: QQC2.AbstractButton.TextBesideIcon
+    property int displayMode: QQC2.AbstractButton.IconOnly
     property int focusPolicy: Qt.StrongFocus
     readonly property real fullButtonHeight: iconTextButtonMetrics.implicitHeight
     readonly property bool mirrored: effectiveLayoutDirection === Qt.RightToLeft
diff --git a/src/Gui/ImageCaptureOverlay.qml b/src/Gui/ImageCaptureOverlay.qml
index 76f963cd640c520e077e48bdb1959bc5f8f084bd..f55e57539691511f7cf779887ea87e6a1bb18856 100644
--- a/src/Gui/ImageCaptureOverlay.qml
+++ b/src/Gui/ImageCaptureOverlay.qml
@@ -180,8 +180,7 @@ MouseArea {
             && !contains(mapFromItem(root, root.mouseX, root.mouseY))
             && !root.pressed
             && !annotations.enabled
-            && !mtbDragHandler.active
-            && !atbDragHandler.active
+            && !toolbarsLayoutDragHandler.active
             && !Selection.rectIntersectsRect(SelectionEditor.handlesRect,
                                              Qt.rect(x, y, width, height))
         Behavior on opacity {
@@ -231,7 +230,7 @@ MouseArea {
                     return Qt.AlignBaseline
                 }
             }
-            readonly property bool normallyVisible: !Selection.empty && !(mainToolBar.visible && mainToolBar.valignment === ssToolTip.valignment)
+            readonly property bool normallyVisible: !Selection.empty && !mainToolBar.visible
             Binding on x {
                 value: contextWindow.dprRound(Selection.horizontalCenter - ssToolTip.width / 2)
                 when: ssToolTip.normallyVisible
@@ -286,231 +285,200 @@ MouseArea {
         Connections {
             target: Selection
             function onEmptyChanged() {
-                if (!Selection.empty
-                    && (mainToolBar.rememberPosition || atbLoader.rememberPosition)) {
-                    mainToolBar.rememberPosition = false
-                    atbLoader.rememberPosition = false
+                if (!Selection.empty) {
+                    toolbarsLayout.rememberPosition = false;
                 }
             }
         }
 
-        // Main ToolBar
-        FloatingToolBar {
-            id: mainToolBar
+        RowLayout {
+            id: toolbarsLayout
+            spacing: Kirigami.Units.smallSpacing
             property bool rememberPosition: false
-            readonly property int valignment: {
+            readonly property int alignment: {
                 if (Selection.empty) {
-                    return 0
+                    return 0;
                 }
-                if (3 * height + topPadding + Kirigami.Units.mediumSpacing
-                    <= SelectionEditor.screensRect.height - SelectionEditor.handlesRect.bottom
-                ) {
-                    return Qt.AlignBottom
-                } else if (3 * height + bottomPadding + Kirigami.Units.mediumSpacing
-                    <= SelectionEditor.handlesRect.top
-                ) {
-                    return Qt.AlignTop
-                } else {
-                    // At the bottom of the inside of the selection rect.
-                    return Qt.AlignBaseline
+                const maxHeight = 2 * height + Kirigami.Units.mediumSpacing;
+                const [min, max] = [
+                    SelectionEditor.screensRect.y,
+                    SelectionEditor.screensRect.y + SelectionEditor.screensRect.height,
+                ];
+                if (SelectionEditor.handlesRect.top - maxHeight >= min) {
+                    return Qt.AlignTop;
                 }
-            }
-            readonly property bool normallyVisible: {
-                let emptyHovered = root.containsMouse && Selection.empty
-                let menuVisible = contextWindow.exportMenu.visible
-                menuVisible |= contextWindow.optionsMenu.visible
-                menuVisible |= contextWindow.helpMenu.visible
-                let pressed = SelectionEditor.dragLocation || annotations.anyPressed
-                return (emptyHovered || !Selection.empty || menuVisible) && !pressed
+                if (SelectionEditor.handlesRect.bottom + maxHeight <= max) {
+                    return Qt.AlignBottom;
+                }
+                return Qt.AlignBaseline
             }
             Binding on x {
                 value: {
-                    const v = Selection.empty ?
-                        (root.width - mainToolBar.width) / 2 + contextWindow.logicalX
-                        : Selection.horizontalCenter - mainToolBar.width / 2
-                    return Math.max(mainToolBar.leftPadding, // min value
-                           Math.min(contextWindow.dprRound(v),
-                                    SelectionEditor.screensRect.width - mainToolBar.width - mainToolBar.rightPadding)) // max value
+                    let x;
+                    if (Selection.empty) {
+                        x = SelectionEditor.screensRect.x + (SelectionEditor.screensRect.width - toolbarsLayout.width) / 2;
+                    } else {
+                        x = Math.max(
+                                SelectionEditor.screensRect.x,
+                                Math.min(
+                                    Selection.horizontalCenter - toolbarsLayout.width / 2,
+                                    SelectionEditor.screensRect.x + SelectionEditor.screensRect.width - toolbarsLayout.width
+                                )
+                            );
+                    }
+                    return contextWindow.dprRound(x);
                 }
-                when: mainToolBar.normallyVisible && !mainToolBar.rememberPosition
+                when: mainToolBar.normallyVisible && !toolbarsLayout.rememberPosition
                 restoreMode: Binding.RestoreNone
             }
             Binding on y {
                 value: {
-                    let v = 0
-                    // put above selection if not enough room below selection
-                    if (mainToolBar.valignment & Qt.AlignTop) {
-                        v = SelectionEditor.handlesRect.top
-                            - mainToolBar.height - mainToolBar.bottomPadding
-                    } else if (mainToolBar.valignment & Qt.AlignBottom) {
-                        v = SelectionEditor.handlesRect.bottom + mainToolBar.topPadding
-                    } else if (mainToolBar.valignment & Qt.AlignBaseline) {
-                        v = Math.min(Selection.bottom, SelectionEditor.handlesRect.bottom - Kirigami.Units.gridUnit)
-                            - mainToolBar.height - mainToolBar.bottomPadding
+                    let y;
+                    if (toolbarsLayout.alignment & Qt.AlignTop) {
+                        y = SelectionEditor.handlesRect.top - toolbarsLayout.height - Kirigami.Units.mediumSpacing;
+                    } else if (toolbarsLayout.alignment & Qt.AlignBottom) {
+                        y = SelectionEditor.handlesRect.bottom + Kirigami.Units.mediumSpacing;
+                    } else if (toolbarsLayout.alignment & Qt.AlignBaseline) {
+                        y = Math.min(Selection.bottom, SelectionEditor.handlesRect.bottom - Kirigami.Units.gridUnit)
+                                - toolbarsLayout.height - Kirigami.Units.mediumSpacing;
                     } else {
-                        v = (mainToolBar.height / 2) - mainToolBar.parent.y
+                        y = SelectionEditor.screensRect.y + toolbarsLayout.height / 2;
                     }
-                    return contextWindow.dprRound(v)
+                    return contextWindow.dprRound(y);
                 }
-                when: mainToolBar.normallyVisible && !mainToolBar.rememberPosition
+                when: mainToolBar.normallyVisible && !toolbarsLayout.rememberPosition
                 restoreMode: Binding.RestoreNone
             }
-            visible: opacity > 0
-            opacity: normallyVisible && Selection.rectIntersectsRect(Qt.rect(x,y,width,height), annotations.viewportRect)
-            Behavior on opacity {
-                NumberAnimation {
-                    duration: Kirigami.Units.longDuration
-                    easing.type: Easing.OutCubic
-                }
-            }
-            layer.enabled: true // improves the visuals of the opacity animation
-            focusPolicy: Qt.NoFocus
-            contentItem: MainToolBarContents {
-                id: mainToolBarContents
-                focusPolicy: Qt.NoFocus
-                displayMode: QQC2.AbstractButton.TextBesideIcon
-                showSizeLabel: mainToolBar.valignment === ssToolTip.valignment
-                imageSize: Selection.alignedSize(Selection.width,
-                                                Selection.height,
-                                                SelectionEditor.devicePixelRatio)
-            }
 
-            DragHandler { // parent is contentItem and parent is a read-only property
-                id: mtbDragHandler
+            DragHandler {
+                id: toolbarsLayoutDragHandler
                 enabled: Selection.empty
-                target: mainToolBar
+                target: toolbarsLayout
                 acceptedButtons: Qt.LeftButton
                 margin: mainToolBar.padding
-                xAxis.minimum: contextWindow.logicalX
-                xAxis.maximum: contextWindow.logicalX + root.width - mainToolBar.width
-                yAxis.minimum: contextWindow.logicalY
-                yAxis.maximum: contextWindow.logicalY + root.height - mainToolBar.height
-                cursorShape: enabled ?
-                    (active ? Qt.ClosedHandCursor : Qt.OpenHandCursor)
-                    : undefined
-                onActiveChanged: if (active && !mainToolBar.rememberPosition) {
-                    mainToolBar.rememberPosition = true
+                xAxis.minimum: SelectionEditor.screensRect.x
+                xAxis.maximum: SelectionEditor.screensRect.x + SelectionEditor.screensRect.width - toolbarsLayout.width
+                yAxis.minimum: SelectionEditor.screensRect.y
+                yAxis.maximum: SelectionEditor.screensRect.y + SelectionEditor.screensRect.height - toolbarsLayout.height
+                cursorShape: enabled ? (active ? Qt.ClosedHandCursor : Qt.OpenHandCursor) : undefined
+                onActiveChanged: {
+                    if (active) {
+                        toolbarsLayout.rememberPosition = true;
+                    }
                 }
             }
-        }
-
-        AnimatedLoader {
-            id: atbLoader
-            property bool rememberPosition: false
-            readonly property int valignment: mainToolBar.valignment & (Qt.AlignTop | Qt.AlignBaseline) ?
-                Qt.AlignTop : Qt.AlignBottom
-            active: visible && mainToolBar.visible
-            onActiveChanged: if (!active && rememberPosition
-                && !contextWindow.annotating) {
-                rememberPosition = false
-            }
-            state: mainToolBar.normallyVisible
-                && contextWindow.annotating ? "active" : "inactive"
 
-            Binding on x {
-                value: {
-                    const min = mainToolBar.x
-                    const target = contextWindow.dprRound(mainToolBar.x + (mainToolBar.width - atbLoader.width) / 2)
-                    const max = mainToolBar.x + mainToolBar.width - atbLoader.width
-                    return Math.max(min, Math.min(target, max))
+            // Main ToolBar
+            FloatingToolBar {
+                id: mainToolBar
+                readonly property bool normallyVisible: {
+                    let emptyHovered = root.containsMouse && Selection.empty
+                    let menuVisible = contextWindow.exportMenu.visible
+                    menuVisible |= contextWindow.optionsMenu.visible
+                    menuVisible |= contextWindow.helpMenu.visible
+                    let pressed = SelectionEditor.dragLocation || annotations.anyPressed
+                    return (emptyHovered || !Selection.empty || menuVisible) && !pressed
                 }
-                when: !atbLoader.rememberPosition
-                restoreMode: Binding.RestoreNone
-            }
-            Binding on y {
-                value: contextWindow.dprRound(atbLoader.valignment & Qt.AlignTop ?
-                    mainToolBar.y - atbLoader.height - Kirigami.Units.mediumSpacing
-                    : mainToolBar.y + mainToolBar.height + Kirigami.Units.mediumSpacing)
-                when: !atbLoader.rememberPosition
-                restoreMode: Binding.RestoreNone
-            }
-
-            DragHandler { // parented to contentItem
-                id: atbDragHandler
-                enabled: Selection.empty
-                acceptedButtons: Qt.LeftButton
-                xAxis.minimum: contextWindow.logicalX
-                xAxis.maximum: contextWindow.logicalX + root.width - atbLoader.width
-                yAxis.minimum: contextWindow.logicalY
-                yAxis.maximum: contextWindow.logicalY + root.height - atbLoader.height
-                cursorShape: enabled ?
-                    (active ? Qt.ClosedHandCursor : Qt.OpenHandCursor)
-                    : undefined
-                onActiveChanged: if (active && !atbLoader.rememberPosition) {
-                    atbLoader.rememberPosition = true
+                visible: opacity > 0
+                opacity: normallyVisible && Selection.rectIntersectsRect(Qt.rect(x,y,width,height), annotations.viewportRect)
+                Behavior on opacity {
+                    NumberAnimation {
+                        duration: Kirigami.Units.longDuration
+                        easing.type: Easing.OutCubic
+                    }
                 }
-            }
-
-            sourceComponent: FloatingToolBar {
-                id: annotationsToolBar
+                layer.enabled: true // improves the visuals of the opacity animation
                 focusPolicy: Qt.NoFocus
-                contentItem: AnnotationsToolBarContents {
-                    id: annotationsContents
-                    displayMode: QQC2.AbstractButton.IconOnly
+                contentItem: MainToolBarContents {
+                    id: mainToolBarContents
                     focusPolicy: Qt.NoFocus
+                    displayMode: QQC2.AbstractButton.IconOnly
+                    showSizeLabel: true
+                    imageSize: Selection.alignedSize(Selection.width,
+                                                    Selection.height,
+                                                    SelectionEditor.devicePixelRatio)
                 }
+            }
 
-                topLeftRadius: optionsToolBar.visible
-                    && optionsToolBar.x === 0
-                    && atbLoader.valignment & Qt.AlignTop ? 0 : radius
-                topRightRadius: optionsToolBar.visible
-                    && optionsToolBar.x === width - optionsToolBar.width
-                    && atbLoader.valignment & Qt.AlignTop ? 0 : radius
-                bottomLeftRadius: optionsToolBar.visible
-                    && optionsToolBar.x === 0
-                    && atbLoader.valignment & Qt.AlignBottom ? 0 : radius
-                bottomRightRadius: optionsToolBar.visible
-                    && optionsToolBar.x === width - optionsToolBar.width
-                    && atbLoader.valignment & Qt.AlignBottom ? 0 : radius
-
-                // Exists purely for cosmetic reasons to make the border of
-                // optionsToolBar that meets annotationsToolBar look better
-                Rectangle {
-                    id: borderBg
-                    z: -1
-                    visible: optionsToolBar.visible
-                    opacity: optionsToolBar.opacity
-                    parent: annotationsToolBar
-                    x: optionsToolBar.x + annotationsToolBar.background.border.width
-                    y: atbLoader.valignment & Qt.AlignTop ?
-                        optionsToolBar.y + optionsToolBar.height : optionsToolBar.y
-                    width: optionsToolBar.width - annotationsToolBar.background.border.width * 2
-                    height: contextWindow.dprRound(1)
-                    color: annotationsToolBar.background.color
-                }
+            AnimatedLoader {
+                id: atbLoader
+                readonly property int valignment: toolbarsLayout.alignment & (Qt.AlignTop | Qt.AlignBaseline) ?
+                    Qt.AlignTop : Qt.AlignBottom
+                active: visible && mainToolBar.visible
+                state: mainToolBar.normallyVisible
+                    && contextWindow.annotating ? "active" : "inactive"
 
-                AnimatedLoader {
-                    id: optionsToolBar
-                    parent: annotationsToolBar
-                    x: {
-                        const targetX = annotationsContents.x
-                            + annotationsContents.checkedButton.x
-                            + (annotationsContents.checkedButton.width - width) / 2
-                        return Math.max(0, // min value
-                               Math.min(contextWindow.dprRound(targetX),
-                                        parent.width - width)) // max value
+                sourceComponent: FloatingToolBar {
+                    id: annotationsToolBar
+                    focusPolicy: Qt.NoFocus
+                    contentItem: AnnotationsToolBarContents {
+                        id: annotationsContents
+                        displayMode: QQC2.AbstractButton.IconOnly
+                        focusPolicy: Qt.NoFocus
                     }
-                    y: atbLoader.valignment & Qt.AlignTop ?
-                        -optionsToolBar.height + borderBg.height
-                        : optionsToolBar.height - borderBg.height
-                    state: if (AnnotationDocument.tool.options !== AnnotationTool.NoOptions
-                        || (AnnotationDocument.tool.type === AnnotationDocument.ChangeAction
-                            && AnnotationDocument.selectedAction.options !== AnnotationTool.NoOptions)
-                    ) {
-                        return "active"
-                    } else {
-                        return "inactive"
+
+                    topLeftRadius: optionsToolBar.visible
+                        && optionsToolBar.x === 0
+                        && atbLoader.valignment & Qt.AlignTop ? 0 : radius
+                    topRightRadius: optionsToolBar.visible
+                        && optionsToolBar.x === width - optionsToolBar.width
+                        && atbLoader.valignment & Qt.AlignTop ? 0 : radius
+                    bottomLeftRadius: optionsToolBar.visible
+                        && optionsToolBar.x === 0
+                        && atbLoader.valignment & Qt.AlignBottom ? 0 : radius
+                    bottomRightRadius: optionsToolBar.visible
+                        && optionsToolBar.x === width - optionsToolBar.width
+                        && atbLoader.valignment & Qt.AlignBottom ? 0 : radius
+
+                    // Exists purely for cosmetic reasons to make the border of
+                    // optionsToolBar that meets annotationsToolBar look better
+                    Rectangle {
+                        id: borderBg
+                        z: -1
+                        visible: optionsToolBar.visible
+                        opacity: optionsToolBar.opacity
+                        parent: annotationsToolBar
+                        x: optionsToolBar.x + annotationsToolBar.background.border.width
+                        y: atbLoader.valignment & Qt.AlignTop ?
+                            optionsToolBar.y + optionsToolBar.height : optionsToolBar.y
+                        width: optionsToolBar.width - annotationsToolBar.background.border.width * 2
+                        height: contextWindow.dprRound(1)
+                        color: annotationsToolBar.background.color
                     }
-                    sourceComponent: FloatingToolBar {
-                        focusPolicy: Qt.NoFocus
-                        contentItem: AnnotationOptionsToolBarContents {
-                            displayMode: QQC2.AbstractButton.IconOnly
+
+                    AnimatedLoader {
+                        id: optionsToolBar
+                        parent: annotationsToolBar
+                        x: {
+                            const targetX = annotationsContents.x
+                                + annotationsContents.checkedButton.x
+                                + (annotationsContents.checkedButton.width - width) / 2
+                            return Math.max(0, // min value
+                                   Math.min(contextWindow.dprRound(targetX),
+                                            parent.width - width)) // max value
+                        }
+                        y: atbLoader.valignment & Qt.AlignTop ?
+                            -optionsToolBar.height + borderBg.height
+                            : optionsToolBar.height - borderBg.height
+                        state: if (AnnotationDocument.tool.options !== AnnotationTool.NoOptions
+                            || (AnnotationDocument.tool.type === AnnotationDocument.ChangeAction
+                                && AnnotationDocument.selectedAction.options !== AnnotationTool.NoOptions)
+                        ) {
+                            return "active"
+                        } else {
+                            return "inactive"
+                        }
+                        sourceComponent: FloatingToolBar {
                             focusPolicy: Qt.NoFocus
+                            contentItem: AnnotationOptionsToolBarContents {
+                                displayMode: QQC2.AbstractButton.IconOnly
+                                focusPolicy: Qt.NoFocus
+                            }
+                            topLeftRadius: atbLoader.valignment & Qt.AlignBottom && x >= 0 ? 0 : radius
+                            topRightRadius: atbLoader.valignment & Qt.AlignBottom && x + width <= annotationsToolBar.width ? 0 : radius
+                            bottomLeftRadius: atbLoader.valignment & Qt.AlignTop && x >= 0 ? 0 : radius
+                            bottomRightRadius: atbLoader.valignment & Qt.AlignTop && x + width <= annotationsToolBar.width ? 0 : radius
                         }
-                        topLeftRadius: atbLoader.valignment & Qt.AlignBottom && x >= 0 ? 0 : radius
-                        topRightRadius: atbLoader.valignment & Qt.AlignBottom && x + width <= annotationsToolBar.width ? 0 : radius
-                        bottomLeftRadius: atbLoader.valignment & Qt.AlignTop && x >= 0 ? 0 : radius
-                        bottomRightRadius: atbLoader.valignment & Qt.AlignTop && x + width <= annotationsToolBar.width ? 0 : radius
                     }
                 }
             }
diff --git a/src/Gui/ImageView.qml b/src/Gui/ImageView.qml
index d7641fcb0c6c31dc679d64962f016a35931b5ade..a3f7c66079a91154f8af7b973e1a31f7f2e01fcc 100644
--- a/src/Gui/ImageView.qml
+++ b/src/Gui/ImageView.qml
@@ -26,7 +26,7 @@ EmptyPage {
     // Used in ViewerWindow::setMode()
     readonly property real minimumWidth: Math.max(
         header.implicitWidth,
-
+        captureOptionsLoader.height + captureOptionsLoader.width,
         annotationsToolBar.implicitWidth + separator.implicitWidth + footerLoader.implicitWidth
     )
     readonly property real minimumHeight: header.implicitHeight
@@ -53,7 +53,7 @@ EmptyPage {
             showNewScreenshotButton: false
             showOptionsMenu: false
             showUndoRedo: contextWindow.annotating
-            displayMode: QQC2.AbstractButton.TextBesideIcon
+            displayMode: QQC2.AbstractButton.IconOnly
         }
     }
 
diff --git a/src/Gui/MainToolBarContents.qml b/src/Gui/MainToolBarContents.qml
index 4013396127d326184db1da30b395215d3e22584e..8704cd908db855da3d74d6c13c9a95438f083721 100644
--- a/src/Gui/MainToolBarContents.qml
+++ b/src/Gui/MainToolBarContents.qml
@@ -26,27 +26,28 @@ ButtonGrid {
         QQC2.ToolTip.delay: Kirigami.Units.toolTipDelay
     }
 
-    AnimatedLoader {
-        id: sizeLabelLoader
+    SizeLabel {
+        id: sizeLabel
         height: root.fullButtonHeight
-        state: root.showSizeLabel && root.imageSize.width > 0 && root.imageSize.height > 0 ?
-            "active" : "inactive"
-        sourceComponent: SizeLabel {
-            size: root.imageSize
-            leftPadding: Kirigami.Units.mediumSpacing + fontMetrics.descent
-            rightPadding: leftPadding
-        }
+        visible: root.showSizeLabel && root.imageSize.width > 0 && root.imageSize.height
+        size: root.imageSize
+        leftPadding: Kirigami.Units.mediumSpacing + fontMetrics.descent
+        rightPadding: leftPadding
     }
 
-    AnimatedLoader {
-        state: root.showUndoRedo ? "active" : "inactive"
-        sourceComponent: UndoRedoGroup {
-            animationsEnabled: root.animationsEnabled
-            buttonHeight: root.fullButtonHeight
-            focusPolicy: root.focusPolicy
-            flow: root.flow
-            spacing: root.spacing
-        }
+    QQC2.ToolSeparator {
+        height: root.flow === Grid.TopToBottom ? implicitWidth : parent.height
+        width: root.flow === Grid.TopToBottom ? parent.width : implicitWidth
+        visible: sizeLabel.visible
+    }
+
+    UndoRedoGroup {
+        visible: root.showUndoRedo
+        animationsEnabled: root.animationsEnabled
+        buttonHeight: root.fullButtonHeight
+        focusPolicy: root.focusPolicy
+        flow: root.flow
+        spacing: root.spacing
     }
 
     ToolButton {

Places

File 2000-toolbars.patch of Package spectacle (Revision 180d56dc52616152104744a2f77adea2)

Places