Bug 407945 - "Files" view shows misaligned folder names
Summary: "Files" view shows misaligned folder names
Status: RESOLVED FIXED
Alias: None
Product: Elisa
Classification: Applications
Component: general (other bugs)
Version First Reported In: 0.4.0
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Matthieu Gallien
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-05-25 15:58 UTC by Patrick Silva
Modified: 2019-05-30 16:14 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments
screenshot (163.54 KB, image/png)
2019-05-25 15:58 UTC, Patrick Silva
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2019-05-25 15:58:52 UTC
Created attachment 120310 [details]
screenshot

SUMMARY
As we can see in the screenshot of elisa 0.4 on Arch Linux,
folder names are misaligned in "Files" view.

STEPS TO REPRODUCE
1. open elisa
2. click "Files" in the left side
3. 

EXPECTED RESULT
folder names are aligned correctly.

SOFTWARE/OS VERSIONS
Operating System: Arch Linux 
KDE Plasma Version: 5.15.5
KDE Frameworks Version: 5.58.0
Qt Version: 5.12.3
Comment 1 Nate Graham 2019-05-26 23:53:05 UTC
Looks like I forgot to update the file browser delegates when I did https://cgit.kde.org/elisa.git/commit/?id=761e3af65347072f12b770529a5effb6b39b6e58
Comment 2 Nate Graham 2019-05-28 20:50:08 UTC
Here's a patch: https://phabricator.kde.org/D21469
Comment 3 Nate Graham 2019-05-30 16:14:07 UTC
Git commit abd7c706965e1b10cd1fd227b828ae1ddb37fcd7 by Nate Graham.
Committed on 30/05/2019 at 16:13.
Pushed by ngraham into branch '0.4'.

Port GridBrowser Delegate UI changes to FileBrowserDelegate too

Summary:
When I did D20581, I forgot to do the File Browser view too. This patch ports the UI
changes done in D20581.

Side note: it feels like these two delegates should really be using the same paramaterized
components rather than mostly sharing the same code which must then be kept in sync. This
one also has no test coverage; even more reason to move towards using the GridBrowserDelegate
for both.

Test Plan:
Tests still pass; UI now looks the same as the grid items in other views
{F6857184}

Reviewers: mgallien, #elisa

Reviewed By: mgallien, #elisa

Differential Revision: https://phabricator.kde.org/D21469

M  +51   -56   src/qml/FileBrowserDelegate.qml

https://commits.kde.org/elisa/abd7c706965e1b10cd1fd227b828ae1ddb37fcd7