Bug 380424 - Adjust grid to align last column with screen edge across different resolutions
Summary: Adjust grid to align last column with screen edge across different resolutions
Status: RESOLVED WORKSFORME
Alias: None
Product: plasmashell
Classification: Plasma
Component: Folder (show other bugs)
Version: 5.10.3
Platform: Neon Linux
: NOR normal
Target Milestone: 1.0
Assignee: Eike Hein
URL: https://github.com/Plasmoner/All-thin...
Keywords:
: 382093 (view as bug list)
Depends on:
Blocks:
 
Reported: 2017-06-01 12:19 UTC by Richard Addison
Modified: 2017-07-07 11:15 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
screenshots illustrating the bug (3.16 MB, application/zip)
2017-06-01 12:19 UTC, Richard Addison
Details
Gap at the right due to tighter folder view item delegates? (830.22 KB, image/png)
2017-06-02 10:22 UTC, Richard Addison
Details
Trying to make it to the little arrow that launches folder view (860.06 KB, image/png)
2017-06-06 11:59 UTC, Richard Addison
Details
... but it's virtually impossible to reach it 'cause the destop toolbox gets triggered in the process :( (859.50 KB, image/png)
2017-06-06 12:08 UTC, Richard Addison
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Richard Addison 2017-06-01 12:19:32 UTC
Created attachment 105819 [details]
screenshots illustrating the bug

Hi,

Attaching 3 shots in a .zip illustrating the bug.

01.png = 5.9.5
02.png = after updating to 5.10.0 (without any further action)
03.png = the best I can do
Comment 1 Richard Addison 2017-06-01 15:41:25 UTC
As an alternative, you can consult this https://github.com/Plasmoner/All-things-plasma/issues/1
Comment 2 Eike Hein 2017-06-02 06:44:39 UTC
This isn't a bug. Plasma 5.10 made the icon delegate more compact based on user feedback.
Comment 3 Richard Addison 2017-06-02 10:22:39 UTC
Created attachment 105842 [details]
Gap at the right due to tighter folder view item delegates?
Comment 4 Eike Hein 2017-06-02 10:24:38 UTC
The gap isn't a new thing - different screen resolutions can fit a number of icon columns more or less neatly. There's currently no attempt made to adjust the grid spacing to the screen resolution.
Comment 5 Richard Addison 2017-06-02 10:26:08 UTC
Okay. I've got it. Thanks.
Comment 6 Richard Addison 2017-06-02 10:34:47 UTC
"attempt made to adjust the grid spacing to the screen resolution"

That would be nice though. Much appreciate your prompt feedback.
Comment 7 Eike Hein 2017-06-02 10:37:21 UTC
Yep, I agree. It's mostly an not-yet-done polish item. I'll edit this ticket to be about that.
Comment 8 Richard Addison 2017-06-02 13:01:30 UTC
That's great news. Feels good to be part of it.
Comment 9 Richard Addison 2017-06-06 11:59:41 UTC
Created attachment 105942 [details]
Trying to make it to the little arrow that launches folder view
Comment 10 Richard Addison 2017-06-06 12:08:35 UTC
Created attachment 105944 [details]
... but it's virtually impossible to reach it 'cause the destop toolbox gets triggered in the process :(
Comment 11 Richard Addison 2017-06-06 12:14:15 UTC
Probably just 2 more redundant screenshots. Just wanna make sure the collateral aspects are fully covered.
Comment 12 Richard Addison 2017-06-06 12:35:44 UTC
All that being said and since reverting the grid to what it was in 5.9.5 is probably out of question, the grid adapting to different resolutions is the only viable/long-term solution.
Comment 13 Richard Addison 2017-07-01 21:25:01 UTC
Reinstalled from neon user 20170629-10:46 build and fully updated. No change. Making the folder view item delegates tighter was a good idea but it crated 3 collateral issues:

1. The gap at the right
2. Conflict with Desktop Toolbox (default position)
3. Conflict with hot corner (default position)

Please check again https://github.com/Plasmoner/All-things-plasma/issues/1

Thank you
Comment 14 Richard Addison 2017-07-01 21:31:25 UTC
Is not 1920 x 1080 a mainstream resolution anymore?
Comment 15 Eike Hein 2017-07-03 04:41:30 UTC
Make sure you used the git master branch.
Comment 16 Jonathan Riddell 2017-07-07 10:48:37 UTC
*** Bug 382093 has been marked as a duplicate of this bug. ***
Comment 17 Richard Addison 2017-07-07 10:56:05 UTC
If it WORKSFORYOU, then it's okay. Thanks.
Comment 18 Richard Addison 2017-07-07 10:59:51 UTC
Should I understand I've run into a dead end? Or are you still considering to fix it in the future?
Comment 19 Richard Addison 2017-07-07 11:03:36 UTC
(In reply to Eike Hein from comment #15)
> Make sure you used the git master branch.

I'm not sure what you mean.
Comment 20 Richard Addison 2017-07-07 11:15:04 UTC
(In reply to Jonathan Riddell from comment #16)
> *** Bug 382093 has been marked as a duplicate of this bug. ***

Thanks. Wasn't sure if If you noticed my report. Silly me. I'm not reproaching anything and I have nothing but good intentions. Kind regards.