Review Request 119986: Dolphin: Rename "Recently Accessed" to "Recently Modified"

kdeuser56 kdeuser56 kdeuser56 at gmail.com
Fri Aug 29 12:54:46 BST 2014


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

(Updated Aug. 29, 2014, 11:54 a.m.)


Review request for Dolphin, Frank Reininghaus and Vishesh Handa.


Changes
-------

corrected the diffs


Repository: kde-baseapps


Description
-------

In dolphin the Section that leads you to timeline claims to show "Recently Accessed", which is not true, it shows "Recently Modified". I think this should be corrected, both in the ui and in the code, since this is very misleading.

Even if it was planned for the future to offer "Recently Accessed" this should be separated anyway from "Recently Modified" since, for a user "Recently Modified" is without any doubt the more important information and should not get lost as a subset of "Recently Accessed".

"Recently Accessed" would anyway cause a lot of trouble with the implementation since a lot of people have disabled atime, so it would not be reliable without a custom daemon watching files for access times.

The diff for the frameworks branch is attatched.


Diffs (updated)
-----

  doc/dolphin/index.docbook 5fe85f5 
  dolphin/src/panels/places/placesitem.h 297f12d 
  dolphin/src/panels/places/placesitem.cpp 539b926 
  dolphin/src/panels/places/placesitemmodel.cpp dc2b95c 
  dolphin/src/panels/places/placespanel.cpp f19fa1e 

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


Testing
-------

compiled and tested


File Attachments (updated)
----------------

frameworks.diff
  https://git.reviewboard.kde.org/media/uploaded/files/2014/08/29/cea6e87a-2f3b-46e8-a89c-41e4cc7f70b4__frameworks-new.diff


Thanks,

kdeuser56 kdeuser56

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20140829/2fd27e64/attachment.htm>


More information about the kfm-devel mailing list