> On May 31, 2011, 9:24 p.m., Kevin Ottens wrote: > > Note that I can't really comment on the activities specific parts, Ivan > > would probably be a better reviewer for that parts. Anyway I found a couple > > of smaller issues which need fixing.
The below have been fixed locally, but I haven't had time to fix the bug I mentioned yet. Ivan, do you have any comments about the implementation? After exams I hope to take a look at this again. - Jeffery ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101348/#review3605 ----------------------------------------------------------- On June 1, 2011, 6:07 a.m., Jeffery MacEachern wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101348/ > ----------------------------------------------------------- > > (Updated June 1, 2011, 6:07 a.m.) > > > Review request for kdelibs, Kevin Ottens, Ivan Čukić, and David Faure. > > > Summary > ------- > > Adds an "Only show in this Activity" option to the KFilePlaces Widget and > support in the underlying model code. Currently only "one activity"/"all > activities" are supported as choices; I think this should be sufficient, and > anything more complicated would be hard to make usable. > > > Diffs > ----- > > kfile/CMakeLists.txt ceae140 > kfile/kfileplaceeditdialog.h d5b030a > kfile/kfileplaceeditdialog.cpp d798b4d > kfile/kfileplacesmodel.h b3dd821 > kfile/kfileplacesmodel.cpp b037084 > kfile/kfileplacesview.cpp 6a343b3 > > Diff: http://git.reviewboard.kde.org/r/101348/diff > > > Testing > ------- > > Tested on Project Neon/Kubuntu Natty. Created several activities, added Place > bookmarks, set them to only show in the current activity, and switched > activities. Everything worked as intended. EDIT: one small known issue - the > OnlyInActivity setting doesn't take when the bookmark is first created; you > have to hit Edit and re-check the box. > > > Thanks, > > Jeffery > >
