> On July 20, 2011, 3:29 p.m., Peter Penz wrote: > > Thanks for the update! > > > > >> would only make sense to push it to the 4.7 branch. > > > What exactly do you mean by 'only'? Isn't 4.7 the branch just > > > about to be released and which will be in all distros until sometime next > > > year? > > > > I meant that this change should only be pushed to the KDE/4.7 and not to > > master as this class will be removed from master around the beginning of > > August (Dolphin 2.0...)
Hi Peter, sorry was busy with uni stuff.. could you please take care of applying this to the correct branch? i don't want to mess up the git.. looking forward to see the new branch evolving... (from its currently somewhat disfunctional state.. really need that file selection 'feature' *g) - Marcel ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101924/#review4903 ----------------------------------------------------------- On July 20, 2011, 2:23 p.m., Marcel Partap wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101924/ > ----------------------------------------------------------- > > (Updated July 20, 2011, 2:23 p.m.) > > > Review request for KDE Base Apps, David Faure and Peter Penz. > > > Summary > ------- > > What was strange that background highlighting and actual item selection were > drawn independently from each other and the bogus highlighting to the left of > the item was not cleared... Now this one was a __REAL__ bitch to get dealt > with, took me hours and hours bashing my head against the shell ^^ > ok now again the viewOptions is not only not the place to turn off background > highlighting, but there it was even tried to ENABLE it :O > turned out this so called QStyle::SH_ItemView_ShowDecorationSelected > documented as "When an item in an item view is selected, also highlight the > branch or other decoration." is hard-coded on by DEFAULT in QCommonStyle and > all inheriting from there so it requires a QProxyStyle to override the > setting. While we have the opportunity, also set > SH_ItemView_ArrowKeysNavigateIntoChildren for added joice of keyboard > navigation (although strange effect comes up when being on a leaf and > pressing Cursor::Right again - but with or without this setting, something > with the selection handler...) > ...now someone owes me CAKE for this one :D > > > Diffs > ----- > > dolphin/src/views/dolphindetailsview.cpp 0ce26df > dolphin/src/views/dolphintreeview.h c037d41 > dolphin/src/views/dolphintreeview.cpp 64b66aa > > Diff: http://git.reviewboard.kde.org/r/101924/diff > > > Testing > ------- > > head-bashing > > > Screenshots > ----------- > > dolphin-treeview-selection-paint-fail > http://git.reviewboard.kde.org/r/101924/s/195/ > > > Thanks, > > Marcel > >
