Created attachment 106232 [details] Gwenview left panel wrongly intended. The version quoted actually is 16.04.3 but this is not available in the menu above. Have a look on the screenshot attached. On the left is Gwenview on the right is Dolphin. The left panels are intended differently.
Thanks for the bug report! This is fixed in GwenView 17.11.70.
Nate, there is no margin. What you might see is the window border?
Oh, you're right. I was comparing two different systems, one with borders and one without.
Looking into writing a patch for this. The big problem seems to be that the everything inside these widgets has zero left padding, which is clearly wrong. We'll want to add some, except for the buttons, where we additionally want to add some left padding to the icon placement, since the icons are right now right up against the button edge, which again is just wrong.
Git commit f0444f600b973ebb7ad226f2ee7007af184fb7e2 by Henrik Fehlauer. Committed on 10/12/2017 at 22:03. Pushed by rkflx into branch 'master'. Fix indentation of icon for left aligned toolbuttons Summary: Gwenview's {nav Operations} sidebar uses custom `QToolButtons` which are wide and left aligned. Special support for this in Breeze was added with 2fb62271d777. However, compared to the KDE4 version there is not enough space between the left button border and the icon: {F5539728} This is fixed by adding the appropriate margin: {F5539730} Note how this replicates exactly the spacing of the toolbuttons in the regular toolbar on the top (which had its icon size set to "small" in this screenshot). Test Plan: Icons on toolbuttons in Gwenview's {nav Operations} sidebar no longer squished all the way to the left, text spacing still looking good. Reviewers: hpereiradacosta Reviewed By: hpereiradacosta Subscribers: ngraham, plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D9281 M +4 -1 kstyle/breezestyle.cpp https://commits.kde.org/breeze/f0444f600b973ebb7ad226f2ee7007af184fb7e2
Git commit f5fd8fa82c6bcaf7066a7031002622458cedd2dd by Henrik Fehlauer. Committed on 10/12/2017 at 22:04. Pushed by rkflx into branch 'Plasma/5.11'. Fix indentation of icon for left aligned toolbuttons Summary: Gwenview's {nav Operations} sidebar uses custom `QToolButtons` which are wide and left aligned. Special support for this in Breeze was added with 2fb62271d777. However, compared to the KDE4 version there is not enough space between the left button border and the icon: {F5539728} This is fixed by adding the appropriate margin: {F5539730} Note how this replicates exactly the spacing of the toolbuttons in the regular toolbar on the top (which had its icon size set to "small" in this screenshot). Test Plan: Icons on toolbuttons in Gwenview's {nav Operations} sidebar no longer squished all the way to the left, text spacing still looking good. Reviewers: hpereiradacosta Reviewed By: hpereiradacosta Subscribers: ngraham, plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D9281 M +4 -1 kstyle/breezestyle.cpp https://commits.kde.org/breeze/f5fd8fa82c6bcaf7066a7031002622458cedd2dd
Git commit 2dfb48a5ce161df548f0a674669c4e3f9385fd06 by Henrik Fehlauer. Committed on 10/12/2017 at 22:07. Pushed by rkflx into branch 'master'. Fix indentation of icon for left aligned toolbuttons Summary: Gwenview's {nav Operations} sidebar uses custom `QToolButtons` which are wide and left aligned. Special support for this in Oxygen was added with b33942690c65. However, compared to the KDE4 version there is not enough space between the left button border and the icon (Screenshots from the matching Diff for Breeze, see D9281): {F5539728} This is fixed by adding the appropriate margin: {F5539730} Note how this replicates exactly the spacing of the toolbuttons in the regular toolbar on the top (which had its icon size set to "small" in this screenshot). Test Plan: Icons on toolbuttons in Gwenview's {nav Operations} sidebar no longer squished all the way to the left, text spacing still looking good. Reviewers: hpereiradacosta Subscribers: ngraham, plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D9282 M +4 -1 kstyle/oxygenstyle.cpp https://commits.kde.org/oxygen/2dfb48a5ce161df548f0a674669c4e3f9385fd06
Git commit 8e51240839744679939a5390dc280f312b20d340 by Henrik Fehlauer. Committed on 10/12/2017 at 22:08. Pushed by rkflx into branch 'Plasma/5.11'. Fix indentation of icon for left aligned toolbuttons Summary: Gwenview's {nav Operations} sidebar uses custom `QToolButtons` which are wide and left aligned. Special support for this in Oxygen was added with b33942690c65. However, compared to the KDE4 version there is not enough space between the left button border and the icon (Screenshots from the matching Diff for Breeze, see D9281): {F5539728} This is fixed by adding the appropriate margin: {F5539730} Note how this replicates exactly the spacing of the toolbuttons in the regular toolbar on the top (which had its icon size set to "small" in this screenshot). Test Plan: Icons on toolbuttons in Gwenview's {nav Operations} sidebar no longer squished all the way to the left, text spacing still looking good. Reviewers: hpereiradacosta Subscribers: ngraham, plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D9282 M +4 -1 kstyle/oxygenstyle.cpp https://commits.kde.org/oxygen/8e51240839744679939a5390dc280f312b20d340
Git commit 92e63330ecdcb1569100036a9a7f5245522fe720 by Henrik Fehlauer, on behalf of Kapil Lamba. Committed on 16/12/2017 at 07:15. Pushed by rkflx into branch 'Applications/17.12'. Correct margin for left sidebar Summary: Sidebar content too close to the left boundary of the window. This patch adds left margin to some of the sidebar contents. Test Plan: Before: {F5524922} After: {F5524923} Reviewers: #kde_applications, ngraham, rkflx Reviewed By: ngraham, rkflx Subscribers: #kde_applications Tags: #kde_applications Differential Revision: https://phabricator.kde.org/D9145 M +3 -1 app/semanticinfocontextmanageritem.cpp M +7 -1 app/sidebar.cpp M +3 -1 app/sidebar.h https://commits.kde.org/gwenview/92e63330ecdcb1569100036a9a7f5245522fe720