-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123738/
-----------------------------------------------------------

(Updated maj 13, 2015, 3:45 e.m.)


Review request for Plasma and Eike Hein.


Changes
-------

Make the if statement easier to read


Repository: plasma-desktop


Description
-------

When we have "Force row settings" and more than one row of items, the items 
start to jump around and it is had to keep track of where each item is. 

The atached patch changes the flow to TopToBottom, in stead of LeftToRight, 
when we have a horizontal layout and "Force row settings", and similarly to 
LeftToRight in vertical layout. (In practice the vertical layout is always one 
column and this patch has no effect)

Here are two videos that describe the problem
First is the row major where taskbar items jump around:
https://youtu.be/8udr2DJKobw

And the second with a patched taskbar where the items jump around a lot less:
https://youtu.be/bk17gnu1ETo


Diffs (updated)
-----

  applets/taskmanager/package/contents/ui/main.qml 98ba7c3 

Diff: https://git.reviewboard.kde.org/r/123738/diff/


Testing
-------

I'm using this patch on all my installations and tried the vertical layout


Thanks,

Kåre Särs

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to