D18938: Show assigned shortcut in drawer action delegates

[Aleix Pol Gonzalez]

apol added a comment.


  Feels a bit in-your-face to me, it can make sense for this application bug for e.g. Discover it would be a bit overwhelming.
  
  I'd prefer keeping it in the tooltip for something generic.

INLINE COMMENTS

> ContextDrawerActionItem.qml:75
>  
> +    QQC2.Label {
> +        Layout.alignment: Qt.AlignVCenter | Qt.AlignRight

this doesn't anchor to the text, won't the text be going over the shortcut if it's too long?

REPOSITORY
  R169 Kirigami

REVISION DETAIL
  https://phabricator.kde.org/D18938

To: hein, mart, apol
Cc: plasma-devel, #plasma, dkardarakos, apol, davidedmundson, mart, hein
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190212/488f9ca1/attachment.html>