Version: Revision: 791773 (using Devel)
Installed from: Compiled sources
How to reproduce the bug:
1) select more than one file
2) apply one tag to the selected files
Result; only one file from the selection is tagged instead of all.
Added Sebastian to CC, should be only a minor issue...
SVN commit 792518 by trueg:
Implemented tagging of multiple files at the same time.
The result is always an intersection of the tags of all files.
Another possibility would be to only change the clicked tag. Feedback?
M +1 -1 metadatawidget.cpp
M +45 -6 tagcloud/resourcetaggingwidget.cpp
M +1 -0 tagcloud/resourcetaggingwidget.h
WebSVN link: http://websvn.kde.org/?view=rev&revision=792518
Sebastian, let me some time to test it. Thanks.
Well, suppose to have 3 tagged files like:
- File1 (holidays, lake)
- File2 (friends)
- File3 (work, lake)
A possible usecase is that the user want to ADD a new tag (for example: "happy") to the 3 files.
If you set the intersection on the tags, you will lost all the existing tags:
(holidays, lake, happy) AND (friends, happy) AND (work, lake, happy) => (happy)
Another usecase: the user select the three files (File1, File2, File3). On the sidebar there are displayed only the tags from the intersection.
But why not the union of the tags? Because I don't know to what file a tag is assigned? Maybe a solution could be to color in black the tags which are on the intersection and gray ot the tags which aren't.
In this way the user can remove a tag from a lot of files easily, even if the tags assigned are not the same on all the files selected.
What do you think?
(I've reopened the bug for this thoughts)
I like your approach. Now we onyl need somebody to implement it. :P
But seriously, I think this should be fixed. Don't know if I find the time to do it, though.
Would it also be possible to show the number of files with each tag?
In the example given above:
- File1 (holidays, lake, happy)
- File2 (friends, happy)
- File3 (work, lake, happy)
The selection of these files would show on the sidebar sorted alphabetically (or by number count):
With happy in black and the other tags in gray.
Sebastian could you give some clues on the way to do it in case someone would find some time to implement it? :p
Not having a reliable way to tag multiple files (due to the "intersection" problem) is kinda frustrating especially now that the search in dolphin makes it really interesting to use!
Is this still a bug? Since i can't find tagging capabilities in dolphin anymore in the latest stable version nor trunk...
*** Bug 251728 has been marked as a duplicate of this bug. ***
Mark, it won`t help you much, but: tagging is working here (kde 4,5,2 and recent trunk too).
*** Bug 273652 has been marked as a duplicate of this bug. ***
Resetting assignee to default as per bug #305719
[Part of the Dolphin Bug Weeks 2012]
The original issue (tags not being applied to all files) was apparently fixed by SC 4.5.2 (maybe earlier). I can confirm that it has not reappeared in Dolphin 2.1/KDE SC 4.9.2 (Kubuntu packages). The tag intersection issue does still exist, though. Not sure if that should be a put into a new report or not.
Reassigning to Nepomuk - the meta data widget isn't part of Dolphin any more, and Vishesh is working on a new implementation inside one of Nepomuk's repositories. Vishesh: Note that the issue which is still buggy is in comment 4.
Git commit 1e3eb42e29571e002f5c09cc81b7beea0121a0d3 by Vishesh Handa.
Committed on 30/12/2012 at 20:47.
Pushed by vhanda into branch 'KDE/4.10'.
WidgetFactory: Properly handle multiple file tagging
When multiple files have been selected then setProperty should not be
used, as the tag widget will only show the common tags, and changing
that list of common tags will result in the not-common tags being
Instead, we use addProperty to add the changed tags. This way the old
tags are preserved. Additionally, we maintain a list of the previous
tags, and calculate the difference to remove the previous tags.
M +21 -3 ui/widgetfactory.cpp
M +1 -0 ui/widgetfactory.h
The "FIXED-IN: 4.11" is not a typo.
I'll see if I can fix this in 4.10, it's not going to be easy.
...but you pushed it to the 4.10 branch.
(In reply to comment #17)
> ...but you pushed it to the 4.10 branch.
I know. But that code isn't used in Dolphin for 4.10. I delayed it too much. The release team is contemplating certain measures like delaying the release and having another RC. Lets see what happens.
I will try to fix this in the old-code as well, but I cannot guarantee that it will make it into 4.10, maybe 4.10.1