Bug 424055 - Pinned launchers add extra unnecessary whitespace to Traditional Task manager, causing improper centering behavior
Summary: Pinned launchers add extra unnecessary whitespace to Traditional Task manager...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Task Manager and Icons-Only Task Manager (show other bugs)
Version: 5.19.2
Platform: openSUSE Linux
: NOR minor
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
: 438371 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-07-10 05:56 UTC by Michael K.
Modified: 2024-05-08 22:04 UTC (History)
7 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.1
Sentry Crash Report:


Attachments
screenshot secondary screen (14.33 KB, image/png)
2020-07-10 05:56 UTC, Michael K.
Details
screenshot primary screen (20.68 KB, image/png)
2020-07-10 05:56 UTC, Michael K.
Details
screencast taskbar (185.59 KB, video/mp4)
2020-07-10 10:26 UTC, Michael K.
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael K. 2020-07-10 05:56:12 UTC
Created attachment 130020 [details]
screenshot secondary screen

SUMMARY
I have two screens and on the second screen I set the taskbar the same as on the first.
The first taskbar is okay, but the window bar of the second taskbar shows so much space to the next plasmoid in the middle. 

STEPS TO REPRODUCE
1. create empty taskbar on the secondary screen
2. configure it the same as the taskbar of the primary screen


OBSERVED RESULT
window bar of each screen has different space to the next plasmoid

EXPECTED RESULT
no space to the next plasmoid on both screens
Operating System: openSUSE Leap 15.2
KDE Plasma Version: 5.18.5
KDE Frameworks Version: 5.71.0
Qt Version: 5.12.7
Kernel Version: 5.3.18-lp152.20.7-default
OS Type: 64-bit
Processors: 8 × Intel® Xeon® CPU E3-1241 v3 @ 3.50GHz
Memory: 15,5 GiB
Comment 1 Michael K. 2020-07-10 05:56:27 UTC
Created attachment 130021 [details]
screenshot primary screen
Comment 2 Méven Car 2020-07-10 09:55:16 UTC
You might have a separator more in the taskbar with the issue between virtual desktop indicator and task list.
Try editing your task bar and remove the separator plasmoid between.
Comment 3 Michael K. 2020-07-10 10:26:52 UTC
Created attachment 130026 [details]
screencast taskbar

No there's no additional spacer between.
Comment 4 Méven Car 2020-07-15 07:19:04 UTC
All right thanks to your screen cast we can see the pinned icons widget is too wide compared to how it should be.
Comment 5 Natalie Clarius 2023-07-31 21:57:38 UTC
*** Bug 438371 has been marked as a duplicate of this bug. ***
Comment 6 Natalie Clarius 2023-07-31 21:58:13 UTC
From the duplicate bug report:  

> I think the problem is `launcherLayoutTasks` in the layout code, which always adds one or more spaces the size of a regular task even when (as in my example) part of it are only launchers.
Comment 7 Natalie Clarius 2023-07-31 21:58:29 UTC
Without launchers the problem does not occur.
Comment 8 Nate Graham 2023-09-18 19:23:17 UTC
*** Bug 472859 has been marked as a duplicate of this bug. ***
Comment 9 Marco Martin 2024-05-06 07:50:17 UTC
Git commit d1b33d1a9dc872e9a2af042152d6ab5ed2617e4e by Marco Martin.
Committed on 06/05/2024 at 07:50.
Pushed by mart into branch 'master'.

Use a GridLayout for taskbar layout

The current javascript layout code is very buggy and error prone,
The normal layouting mechanism of GridLayout is much more reliable
and does for faster and more readable code
Related: bug 485223

M  +2    -2    applets/taskmanager/package/contents/ui/ContextMenu.qml
M  +4    -4    applets/taskmanager/package/contents/ui/GroupDialog.qml
M  +1    -17   applets/taskmanager/package/contents/ui/MouseHandler.qml
M  +109  -28   applets/taskmanager/package/contents/ui/Task.qml
M  +28   -14   applets/taskmanager/package/contents/ui/TaskList.qml
D  +0    -246  applets/taskmanager/package/contents/ui/code/layout.js
A  +126  -0    applets/taskmanager/package/contents/ui/code/layoutmetrics.js
M  +51   -49   applets/taskmanager/package/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/d1b33d1a9dc872e9a2af042152d6ab5ed2617e4e