Review Request: Add Activity Awareness to KFilePlaces* Widget (OnlyInActivity)

Jeffery MacEachern j.maceachern at gmail.com
Fri May 13 03:26:51 BST 2011


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

(Updated May 13, 2011, 2:26 a.m.)


Review request for kdelibs, Kevin Ottens and David Faure.


Changes
-------

Cleaned up some whitespace and other stylistic errors.


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 (updated)
-----

  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.


Thanks,

Jeffery

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110513/30e4b746/attachment.htm>


More information about the kde-core-devel mailing list