----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3375/#review4637 -----------------------------------------------------------
so essentially it removes any sorting within the desktop sorting. i'm fine with that change (though perhaps others will now complain ;), though i have some thoughts below. trunk/KDE/kdebase/workspace/libs/taskmanager/strategies/desktopsortingstrategy.cpp <http://reviewboard.kde.org/r/3375/#comment4158> so window groups will shuffle randomly when windows leave or join or anything else that may cause winIds() to shift order. seems like trading one issue for another. i wonder if it wouldn't be nicer to have a stable way to identify AbstractGroupableTasks (a QUuid? or an integer based sequence?) for such occasions. that would also make things such as leftWinIdIsValid unecessary. the winId is just as random to the user; the only downside i can see is a small amount more memory consumption. trunk/KDE/kdebase/workspace/libs/taskmanager/strategies/desktopsortingstrategy.cpp <http://reviewboard.kde.org/r/3375/#comment4157> } else { trunk/KDE/kdebase/workspace/libs/taskmanager/strategies/desktopsortingstrategy.cpp <http://reviewboard.kde.org/r/3375/#comment4161> can be just "else if (leftWinIdIsValid)" since we know both aren't due to the initial if. trunk/KDE/kdebase/workspace/libs/taskmanager/strategies/desktopsortingstrategy.cpp <http://reviewboard.kde.org/r/3375/#comment4160> why storing in a temp var which is returned? - Aaron On 2010-03-23 20:48:15, Dmitry Suzdalev wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/3375/ > ----------------------------------------------------------- > > (Updated 2010-03-23 20:48:15) > > > Review request for Plasma. > > > Summary > ------- > > This patch fixes a sorting order for "Sort by Desktop" mode of taskmanager > lib. > > Summary: > When in "Sort by Desktop" mode, sort by_desktop+by_winId, instead of > by_desktop+by_winTitle as it is now. > > More details: > Currently in "by desktop" sorting mode tasks are sorted by desktop and then > by name. > This leads to inconvenient things, here are some examples: > > - I have a browser with several tabs open. Whenever I change a tab, browser > changes window title. > This makes task jump in a task bar from one position to another while I'm > simply changing tabs. > - I have a 'konsole' window and as I do 'cd onedir', 'cd zletter_dir', etc, > title is changed, task jumps > - Some other situations caused this too, don't recall, but you got the idea :) > > What I've done: > Instead of sorting by name, i made it to sort by winId. Tasks without winId > are sorted out to the end of the list > and sorted by name there. Typically these are the startup-in-progress items. > > As a side effect this fixes bug https://bugs.kde.org/show_bug.cgi?id=219528 > which has the same roots: > invalid sorting order due to wrong comparison of regular items with "starting > up" items. > > Long description, short patch ;) > > > This addresses bug 219528. > https://bugs.kde.org/show_bug.cgi?id=219528 > > > Diffs > ----- > > > trunk/KDE/kdebase/workspace/libs/taskmanager/strategies/desktopsortingstrategy.cpp > 1105743 > > Diff: http://reviewboard.kde.org/r/3375/diff > > > Testing > ------- > > Tested on trunk. Task items retain their sort order, not reacting to title > changes, charming :) > > It affects only task applets with sort mode set to "by desktop" > > > Thanks, > > Dmitry > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
