filipf added a comment.

  Agreed, let's not change the padding.
  
  One issue that remains is that headings don't work well with RTL layouts but 
I can't quite figure out a solution yet.

INLINE COMMENTS

> ExpandedRepresentation.qml:55
>                  } else if (activeApplet && hiddenItemsView.visible) {
> -                    return hiddenItemsView.iconColumnWidth + 
> units.largeSpacing;
> +                    return hiddenItemsView.width + separator.width + 
> units.smallSpacing / 2;
>  

maybe this could be somehow more precise

REPOSITORY
  R120 Plasma Workspace

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

To: filipf, #plasma, #vdg
Cc: ngraham, kmaterka, mart, GB_2, plasma-devel, LeGast00n, The-Feren-OS-Dev, 
jraleigh, fbampaloukas, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra

Reply via email to