D11065: [Active Window Control] Slightly fix vertical alignment of window name

Radek HuĊĦek noreply at phabricator.kde.org
Mon Mar 5 19:14:00 UTC 2018


Pitel added a comment.


  Would it be better to move global menu 1px down instead?
  
                     PlasmaComponents.Label {
                         id: appmenuButtonTitle
    -                    anchors.centerIn: appmenuButtonBackground
    +                    anchors.top: appmenuButtonBackground.top
    +                    anchors.bottom: appmenuButtonBackground.bottom
    +                    verticalAlignment: Text.AlignVCenter
    +                    anchors.horizontalCenter: appmenuButtonBackground.horizontalCenter
                         font.pixelSize: fontPixelSize * plasmoid.configuration.appmenuButtonTextSizeScale
                         text: activeMenu.replace('&', '')
                         font.weight: appmenuFontBold ? Font.Bold : theme.defaultFont.weight
  
  It also fixes the issue (although I liked the 1px higher a bit better in this specific case).
  
  F5742205: Screenshot_20180305_200905.png <https://phabricator.kde.org/F5742205>
  
  Anyway the root of the problem seems to be that each of them was centered in a different way.

REPOSITORY
  R884 Active Window Control Applet for Plasma

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

To: Pitel, #plasma, martinkostolny
Cc: broulik, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180305/0d392a73/attachment.html>


More information about the Plasma-devel mailing list