Give "+" tool button a proper a11y role and an RTL-friendly modal menu
This commit is contained in:
@@ -55,6 +55,7 @@ RowLayout {
|
|||||||
|
|
||||||
QQC2.ToolButton {
|
QQC2.ToolButton {
|
||||||
id: menuButton
|
id: menuButton
|
||||||
|
Accessible.role: Accessible.ButtonMenu
|
||||||
display: QQC2.AbstractButton.IconOnly
|
display: QQC2.AbstractButton.IconOnly
|
||||||
checkable: true
|
checkable: true
|
||||||
action: Kirigami.Action {
|
action: Kirigami.Action {
|
||||||
@@ -62,10 +63,10 @@ RowLayout {
|
|||||||
icon.name: "irc-join-channel"
|
icon.name: "irc-join-channel"
|
||||||
onTriggered: {
|
onTriggered: {
|
||||||
if (Kirigami.isMobile) {
|
if (Kirigami.isMobile) {
|
||||||
let menu = mobileMenu.createObject();
|
const menu = mobileMenu.createObject();
|
||||||
menu.open();
|
menu.open();
|
||||||
} else {
|
} else {
|
||||||
let menu = desktopMenu.createObject(menuButton, {y: menuButton.height});
|
const menu = desktopMenu.createObject(menuButton);
|
||||||
menu.closed.connect(menuButton.toggle)
|
menu.closed.connect(menuButton.toggle)
|
||||||
menu.open();
|
menu.open();
|
||||||
}
|
}
|
||||||
@@ -80,6 +81,12 @@ RowLayout {
|
|||||||
Component {
|
Component {
|
||||||
id: desktopMenu
|
id: desktopMenu
|
||||||
QQC2.Menu {
|
QQC2.Menu {
|
||||||
|
x: mirrored ? parent.width - width : 0
|
||||||
|
y: parent ? parent.height : 0
|
||||||
|
|
||||||
|
modal: true
|
||||||
|
dim: false
|
||||||
|
|
||||||
QQC2.MenuItem {
|
QQC2.MenuItem {
|
||||||
action: exploreAction
|
action: exploreAction
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user