blob: 59bb60c6c347a3737a6be0156514461fd28fc9d7 (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
|
From 939273025ed92246b5e6d1e3a944a8c88bbc682b Mon Sep 17 00:00:00 2001
From: ivan tkachenko <me@ratijas.tk>
Date: Fri, 22 Jul 2022 21:27:04 +0300
Subject: [PATCH] ToolTip: Sync to PC3: Strip out ampersands
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Easily reproducible broken example: put a notes applet on a desktop,
hover over its "Configure…" button (it should be fine so far), then
right click it, press Escape (or otherwise close the context menu, and
hover again: now the tooltip spells "&Configure…" because for some
reason a mnemonic was added during this time.
---
org.kde.desktop/ToolTip.qml | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/org.kde.desktop/ToolTip.qml b/org.kde.desktop/ToolTip.qml
index 40dfcbd..3a03cc5 100644
--- a/org.kde.desktop/ToolTip.qml
+++ b/org.kde.desktop/ToolTip.qml
@@ -13,7 +13,7 @@ import QtQuick.Layouts 1.15
import org.kde.kirigami 2.12 as Kirigami
T.ToolTip {
- id: controlRoot
+ id: control
Kirigami.Theme.colorSet: Kirigami.Theme.Tooltip
Kirigami.Theme.inherit: false
@@ -41,9 +41,11 @@ T.ToolTip {
closePolicy: T.Popup.CloseOnEscape | T.Popup.CloseOnPressOutsideParent | T.Popup.CloseOnReleaseOutsideParent
contentItem: Controls.Label {
- text: controlRoot.text
+ // Strip out ampersands right before non-whitespace characters, i.e.
+ // those used to determine the alt key shortcut
+ text: control.text.replace(/&(?=\S)/g, "")
wrapMode: Text.WordWrap
- font: controlRoot.font
+ font: control.font
Kirigami.Theme.colorSet: Kirigami.Theme.Tooltip
color: Kirigami.Theme.textColor
}
--
GitLab
|