Summary: | Album-sorting gone from menus [patch] | ||
---|---|---|---|
Product: | [Applications] digikam | Reporter: | S. Burmeister <sven.burmeister> |
Component: | Usability-Menus | Assignee: | Digikam Developers <digikam-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | caulier.gilles, frederic.coiffier, marcel.wiesweg, mohammed.ahmed.anwer, omidlink+bugs.kde.org, rke, wstephenson |
Priority: | NOR | Flags: | languitar:
Junior_Jobs+
|
Version: | 3.5.0 | ||
Target Milestone: | --- | ||
Platform: | openSUSE | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | 4.0.0 | |
Sentry Crash Report: | |||
Bug Depends on: | |||
Bug Blocks: | 126116, 218023 | ||
Attachments: |
added an item to the view menu that sorts albums in the side bar by name "ascending & descending"
Patch for Digikam to add Album Sorting by name (Ascending and Descending) to the View menu |
Description
S. Burmeister
2010-02-18 15:59:29 UTC
Yes, this indeed missing since the model view port since we didn't find a simple solution to implement this, although it is possible. Can you explain a bit more for which purpose you need this feature. Maybe we can use this as an opportunity to also improve the general usability. Each album has properties, such as date, name or category. Some people like to have the most recent albums on top other the oldest. It's basically like a file/folder list in dolphin where I can sort by properties and group by properties. I only use one collection, so I'm not sure whether it makes sense to be able to sort collection-less. Sorting, when keeping the hierarchy, is no problem at all. What is bugging is is the grouping - in the old versions, all albums were put in a flat list and new parent entries were inserted for each category. Currently this regrouping is virtually impossible to do with model/view (a class to achieve this is being developed for akonadi but not yet in kdelibs). Just stumbled over bug 126116. If we fix this bug we should also resolve the other one. Should not be too much additional work. In my view the previous option to sort albums via categories was an important option to structure an increasing amount of albums. With the current solution based on a flat list the album overview has become a bit confusing. In my case, I sorted all my albums by date. I think that the calendar view is different as we can't see Album names in the tree. Moreover, the album date defined in "Properties" is become useless in this current version as it was essentially used for sorting, isn't it ? Sorting albums seems to be gone from the menu... have we removed this feature on purpose? See #1 and #3. Regarding #3, is there some news about this class in akonadi and its inclusion in kdelibs ? Have you another plan for near future ? Since this feature had disappeared, I have to rename all my Albums (which is the opposite of the original purpose of Digikam). *** This bug has been confirmed by popular vote. *** *** Bug 291544 has been marked as a duplicate of this bug. *** Some technical information from Marcel taken from a current thread in digikam-users mailing list : The task is not too difficult. Sorting is implemented in AlbumFilterModel::lessThan. This method does not need to be changed if the sort role is adjusted properly. For that, we'll need a special AlbumDateRole in AlbumModel, and a reimplementation of albumData for that class. In the UI, DigikamView / leftsidebarwidgets need to set the sort role on the album model (from albummodelcollection) which is used for the left side bar. Created attachment 79864 [details]
added an item to the view menu that sorts albums in the side bar by name "ascending & descending"
My mentor [Islam al Wazery] asked me to work on this bug,
He explained and clarified the different parts of the source code of Digikam to me when we met.
then I started to work on the bug which is actually a feature to be added to Digikam's view menu.
I added a new Item to the menu called "Sort Albums By Name" which is branched after that to
"ascending & descending" sort orders.
after my exams I will work on adding some sorting options for albums "By Name, By Size , By Path ,....".
thats all.
The most important is to be able to sorting them by date (thanks to the date which is assigned in the property of each album) Gilles, can you review and apply the patch, if not I will do that in Thursday. On 5/13/13, Mohamed <mohammed.ahmed.anwer@gmail.com> wrote: > https://bugs.kde.org/show_bug.cgi?id=227530 > > --- Comment #13 from Mohamed <mohammed.ahmed.anwer@gmail.com> --- > Created attachment 79864 [details] > --> https://bugs.kde.org/attachment.cgi?id=79864&action=edit > added an item to the view menu that sorts albums in the side bar by name > "ascending & descending" > > My mentor [Islam al Wazery] asked me to work on this bug, > He explained and clarified the different parts of the source code of Digikam > to > me when we met. > then I started to work on the bug which is actually a feature to be added > to > Digikam's view menu. > I added a new Item to the menu called "Sort Albums By Name" which is > branched > after that to > "ascending & descending" sort orders. > after my exams I will work on adding some sorting options for albums "By > Name, > By Size , By Path ,....". > thats all. > > -- > You are receiving this mail because: > You are the assignee for the bug. > _______________________________________________ > Digikam-devel mailing list > Digikam-devel@kde.org > https://mail.kde.org/mailman/listinfo/digikam-devel > Islam, Do you review this patch ? Gilles Caulier Mohamed, I review you patch, and it look mostly uncomplete. For example, i see slot named "S" in signal/slot connection. I think you attach and old version of patch in this file. Also, it do not compile against git/master. It must be updated. Thanks in advance Gilles Caulier Mr Gilles, Yes, I wil update it soon Thanks, Mohamed Anwer Mohamed, I'm always waiting your updated patch here... Gilles Caulier Created attachment 83274 [details]
Patch for Digikam to add Album Sorting by name (Ascending and Descending) to the View menu
Here is the patch Mr Gilles ,, sorry for being late
Mohamed, Patch checked and tested. It sound fine, but it miss real improvements to sort albums as by date. Note : Sorting album by name, is not really an improvement since this option is already available in album tree-view header. Just click one time album are sorted in descending mode, and one another time, albums are sorted in ascending mode... Gilles Caulier *** Bug 218022 has been marked as a duplicate of this bug. *** |