D6150: [Notes Applet] Show formatting options when it has focus

David Edmundson noreply at phabricator.kde.org
Thu Jun 8 10:29:04 UTC 2017


davidedmundson added a comment.


  Seems sensible it doesn't save any space as we anchor the text above the button anyway.

INLINE COMMENTS

> main.qml:238
>  
> -        PlasmaComponents.ToolButton {
> -            id: toggleFormatBarButton
> -            tooltip: i18n("Toggle text format options")
> -            iconSource: "draw-text"
> -            checkable: true
> -            Accessible.name: tooltip
> -            visible: fontButtons.showFormatButtons
> -        }
> -
>          Row {
>              id: formatButtonsRow

probably don't need this Row then, it's just there to make toggling easier?

> main.qml:242
> +            // show format buttons if TextField or any of the buttons have focus
> +            opacity: {
> +                if (!fontButtons.showFormatButtons) {

just

root.Plasmoid.activeFoocus

after D6512

REPOSITORY
  R114 Plasma Addons

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

To: broulik, #plasma, #vdg
Cc: davidedmundson, plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170608/c893fc89/attachment.html>


More information about the Plasma-devel mailing list