filipf added a comment.
The latest version already works with RTL without the `fillWidth` code being needed. So as far as I can tell the diff you posted only effectively removes the margins from headings. But I think we should keep them, i.e., align those headings with the container (@GB_2 already mentioned this : D24720#548745 <https://phabricator.kde.org/D24720#548745>) F7642540: image.png <https://phabricator.kde.org/F7642540> But in the case of RTL perhaps the headings should be on the other side actually, same as before? F7642559: image.png <https://phabricator.kde.org/F7642559> 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
