Review Request 123738: Use column major in the taskbar when "Force row settings" is set

Kåre Särs kare.sars at iki.fi
Wed May 20 19:45:37 UTC 2015


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

(Updated May 20, 2015, 7:45 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma and Eike Hein.


Changes
-------

Submitted with commit c685b9ce1f0d567b2c2c2e27a546506d29b41114 by Kåre Särs to branch master.


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
-----

  applets/taskmanager/package/contents/ui/ConfigGeneral.qml 36dd134 
  applets/taskmanager/package/contents/ui/main.qml 98ba7c3 

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


Testing
-------

I have applied this patch to the system installed plasma 5.3 installation.


Thanks,

Kåre Särs

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150520/8d2505d8/attachment.html>


More information about the Plasma-devel mailing list