apol accepted this revision.
apol added a comment.
This revision is now accepted and ready to land.


  Otherwise it's good to go.

INLINE COMMENTS

> ActionListItem.qml:44
>  
> -    ToolTip.visible: hovered
> -    ToolTip.text: action.shortcut ? action.shortcut : p0.nativeText
> +        readonly property var tooltip: QQC2.ToolTip {
> +            text: action.shortcut ? action.shortcut : p0.nativeText

It has an indentation too much.

REPOSITORY
  R134 Discover Software Store

BRANCH
  actionlistitem-make-qml-imports-and-tooltip-consistent (branched from master)

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

To: GB_2, #plasma, #vdg, apol
Cc: apol, #vdg, plasma-devel, #plasma, jraleigh, GB_2, ragreen, Pitel, ZrenBot, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart

Reply via email to