Differential D21815 Diff 59884 sddm-theme/SessionButton.qml

Changeset View

Standalone View

View Options

sddm-theme/SessionButton.qml

Show All 11 Lines
12 * GNU General Public License for more details 12​ * GNU General Public License for more details
13 * 13​ *
14 * You should have received a copy of the GNU Library General Public 14​ * You should have received a copy of the GNU Library General Public
15 * License along with this program; if not, write to the 15​ * License along with this program; if not, write to the
16 * Free Software Foundation, Inc., 16​ * Free Software Foundation, Inc.,
17 * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. 17​ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
18 */ 18​ */
19 19​
20import QtQuick 2.2 20​import QtQuick 2.6
21 21​
22import org.kde.plasma.core 2.0 as PlasmaCore 22​import org.kde.plasma.core 2.0 as PlasmaCore
23import org.kde.plasma.components 2.0 as PlasmaComponents 23​import org.kde.plasma.components 3.0 as PlasmaComponents
davidedmundsonUnsubmitted
Not Done

Note that (confusingly) Plasma components 2 is QQC1
Plasma components3 is QQC2

It might fix the menu issue

davidedmundson: Note that (confusingly) Plasma components 2 is QQC1 Plasma components3 is QQC2 It might fix…
cblackAuthorUnsubmitted
Not Done

The menu property seems to be removed in PC3

cblack: The `menu` property seems to be removed in PC3
cblackAuthorUnsubmitted
Not Done

Additionally, ToolButton still seems to use QQC1 even in PC3.

cblack: Additionally, [[ https://api.kde.org/frameworks/plasma…
24 24​
25import QtQuick.Controls 1.3 as QQC 25​import QtQuick.Controls 2.5 as QQC2
26 26​
27PlasmaComponents.ToolButton { 27​Item {
davidedmundsonUnsubmitted
Not Done

Why the wrapper?

davidedmundson: Why the wrapper?
cblackAuthorUnsubmitted
Not Done

QML files can only have one root, and the ToolButton's menu property doesn't accept QQC2 menus, meaning they have to be separated.

cblack: QML files can only have one root, and the ToolButton's `menu` property doesn't accept QQC2…
filipfUnsubmitted
Not Done

It seems to be running fine without the Item wrapper for me. What I did though was but the QQC2 menu inside the button, I didn't split them up.

filipf: It seems to be running fine without the Item wrapper for me. What I did though was but the QQC2…
28 id: root 28​ id: root
29 property int currentIndex: -1 29​ implicitHeight: toolBtn.implicitHeight
30​ implicitWidth: toolBtn.implicitWidth
30 31​
31 implicitWidth: minimumWidth 32​ PlasmaComponents.ToolButton {
33​ id: toolBtn
34​ property int currentIndex: -1
davidedmundsonUnsubmitted
Not Done

Implicit height should be propagated upwards, not height.

davidedmundson: Implicit height should be propagated upwards, not height.
davidedmundsonUnsubmitted
Not Done

I meant

implicitWidth: toolBtn.implicitWidth

width by default is bound to the implicitWidth so you don't need to set it
but more importantly you still have the data of the size something wants to be after someone else (a layout) sets the size

davidedmundson: I meant implicitWidth: toolBtn.implicitWidth width by default is bound to the implicitWidth…
32 35​
33 visible: menu.items.length > 1 36​ visible: sessionMenu.count > 1
34 37​
35 text: i18nd("plasma_lookandfeel_org.kde.lookandfeel", "Desktop Session: %1", instantiator.objectAt(currentIndex).text || "") 38​ text: i18nd("plasma_lookandfeel_org.kde.lookandfeel", "Desktop Session: %1", instantiator.objectAt(currentIndex).name || "Error")
36 39​
37 font.pointSize: config.fontSize 40​ font.pointSize: config.fontSize
38 41​
39 Component.onCompleted: { 42​ Component.onCompleted: {
40 currentIndex = sessionModel.lastIndex 43​ currentIndex = sessionModel.lastIndex
41 } 44​ }
42 45​
43 menu: QQC.Menu { 46​ onClicked: {
44 id: menu 47​ sessionMenu.popup(x, y)
filipfUnsubmitted
Not Done

While we're doing this, can you perhaps make the menu pop up *above* the button?

filipf: While we're doing this, can you perhaps make the menu pop up *above* the button?
cblackAuthorUnsubmitted
Not Done

It seems that changing the y value doesn't affect its y position for whatever reason.

cblack: It seems that changing the `y` value doesn't affect its y position for whatever reason.
45 style: BreezeMenuStyle {} 48​ }
49​ }
50​
51​ QQC2.Menu {
52​ id: sessionMenu
53​ property int largestWidth: 9999
54​
55​ Component.onCompleted: {
56​ var trueWidth = 0;
57​ for (var i = 0; i < sessionMenu.count; i++) {
58​ trueWidth = Math.max(trueWidth, sessionMenu.itemAt(i).textWidth)
59​ }
60​ sessionMenu.largestWidth = trueWidth
61​ }
62​
63​ background: Rectangle {
64​ implicitHeight: 40
65​ implicitWidth: sessionMenu.largestWidth
filipfUnsubmitted
Not Done

This can be up for debate, but I sort of liked the older behavior more when the menu's width would be determined by the size of the ToolButton at a particular point.

filipf: This can be up for debate, but I sort of liked the older behavior more when the menu's width…
66​ color: PlasmaCore.ColorScope.backgroundColor
67​ border.color: Qt.tint(PlasmaCore.ColorScope.textColor, Qt.rgba(color.r, color.g, color.b, 0.7))
68​ border.width: 1
69​ }
70​
46 Instantiator { 71​ Instantiator {
47 id: instantiator 72​ id: instantiator
48 model: sessionModel 73​ model: sessionModel
49 onObjectAdded: menu.insertItem(index, object) 74​ onObjectAdded: sessionMenu.addItem( object )
50 onObjectRemoved: menu.removeItem( object ) 75​ onObjectRemoved: sessionMenu.removeItem( object )
51 delegate: QQC.MenuItem { 76​ delegate: QQC2.MenuItem {
77​ id: menuItem
78​ property string name: model.name
79​
80​ property real textWidth: text.contentWidth + 20
81​ implicitWidth: text.contentWidth + 20
82​ implicitHeight: text.contentHeight * 1.6
83​
84​ contentItem: QQC2.Label {
85​ id: text
52 text: model.name 86​ text: model.name
53 onTriggered: { 87​ font: menuItem.font
filipfUnsubmitted
Not Done

Why do we need to set this?

filipf: Why do we need to set this?
54 root.currentIndex = model.index 88​ color: PlasmaCore.ColorScope.textColor
89​ horizontalAlignment: Text.AlignLeft
90​ verticalAlignment: Text.AlignVCenter
91​ elide: Text.ElideRight
92​ }
93​
94​ background: Rectangle {
95​ color: menuItem.highlighted ? PlasmaCore.ColorScope.highlightColor : "transparent"
96​ MouseArea {
97​ anchors.fill: parent
98​ hoverEnabled: true
99​ onContainsMouseChanged: menuItem.highlighted = containsMouse
100​ onClicked: {
101​ sessionMenu.dismiss()
102​ toolBtn.currentIndex = model.index
103​ }
104​ }
105​ }
106​ }
107​ }
108​
109​ enter: Transition {
110​ NumberAnimation {
111​ property: "opacity"
112​ from: 0
113​ to: 1
114​ duration: 150
115​ }
55 } 116​ }
117​ exit: Transition {
118​ NumberAnimation {
119​ property: "opacity"
120​ from: 1
121​ to: 0
122​ duration: 150
56 } 123​ }
57 } 124​ }
58 } 125​ }
59} 126​}