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
