Differential D27149 Diff 74981 applets/taskmanager/package/contents/ui/TextWrapper.qml

Changeset View

Standalone View

View Options

applets/taskmanager/package/contents/ui/TextWrapper.qml

  • This file was added.
1​/*
2​* Copyright 2020 by Tranter Madi <trmdi@yandex.com>
3​*
4​* This program is free software; you can redistribute it and/or modify
5​* it under the terms of the GNU Library General Public License as
6​* published by the Free Software Foundation; either version 2, or
7​* (at your option) any later version.
8​*
9​* This program is distributed in the hope that it will be useful,
10​* but WITHOUT ANY WARRANTY; without even the implied warranty of
11​* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12​* GNU Library General Public License for more details
13​*
14​* You should have received a copy of the GNU Library General Public
15​* License along with this program; if not, write to the
16​* Free Software Foundation, Inc.,
17​* 51 Franklin Street, Fifth Floor, Boston, MA 2.010-1301, USA.
18​*/
19​
20​import QtQuick 2.6
21​
22​Item {
23​ id: textWrapper
24​
25​ property var textItem
broulikUnsubmitted
Not Done

Define it more specifically as property Text textItem

broulik: Define it more specifically as `property Text textItem`
26​ readonly property alias containsMouse: mouseItem.containsMouse
27​ property int hoveringTime: 0
28​ readonly property bool longHovering: hoveringTime >= 500
29​
30​ clip: textItem.elide === Text.ElideNone
31​
32​ onContainsMouseChanged: {
33​ hoveringTime = 0
34​ if (containsMouse && width <= textItem.implicitWidth) {
35​ timer.start()
36​ } else {
37​ timer.stop()
38​ }
39​ }
40​
41​ onLongHoveringChanged: {
42​ if (longHovering) {
43​ state = "longHovering"
44​ timer.stop()
45​ } else {
46​ state = ""
47​ }
48​ }
49​
50​ MouseArea {
broulikUnsubmitted
Not Done

You can just make the textWrapper be a MouseArea, saves you having an extra area inside

broulik: You can just make the `textWrapper` be a `MouseArea`, saves you having an extra area inside
51​ id: mouseItem
52​ anchors.fill: parent
53​ hoverEnabled: true
54​ }
55​
56​ Timer {
57​ id: timer
58​ running: false; interval: 100; repeat: true;
broulikUnsubmitted
Not Done

You can simplify this long duration handling a bit:

Timer {
    interval: 500
    running: area.containsMouse
    onTriggered: {
        area.state = "longHovering"
    }
}

and then all you need is

onContainsMouseChanged: {
    if (!containsMouse) {
        area.state = "";
    }
}

I *think* you could even do it more declaratively by having the State do

when: area.containsMouse && !timer.running

but since property evaluation order is non-deterministic, it might briefly enter the state on hover

broulik: You can simplify this long duration handling a bit: ``` Timer { interval: 500 running…
59​ onTriggered: {
60​ parent.hoveringTime += 100
61​ }
62​ }
63​
64​ states: [
65​ State {
66​ name: ""
67​ PropertyChanges { target: textItem; x: 0 }
68​ },
69​ State {
70​ name: "longHovering"
71​ PropertyChanges { target: textItem; x: textWrapper.width - textItem.implicitWidth }
72​ }
73​ ]
74​
75​ transitions: Transition {
76​ to: "longHovering"
77​ NumberAnimation {
78​ target: textItem; properties: "x";
79​ easing.type: Easing.Linear;
80​ duration: (textItem.implicitWidth - textWrapper.width)*25
81​ }
82​ }
83​
84​}