Patch: adjust grouping behaviour

Simon Frei freisim93 at gmail.com
Thu Nov 3 12:00:15 GMT 2016


Currently when a grouped image is selected all operations (tagging,
renaming, ...) are only applied to the head of the group. This is not
consistent with the options to display items: In iconview, there is
grouped and ungrouped, in tableview ungrouped, grouped and tree. So I
have control over whether I want to display images individually or as
groups. However this control does not extend to actions.

The attached patches change the behaviour:
When images are grouped or not expanded in treeview, actions apply to
the selected images and the grouped images. In Ungrouped view or when
images are expanded in treeview, actions apply only to the selected images.

In the process I uncovered some more or less related issues, that is why
I have split the patch up. Every single one is functional.
"allchanges.diff" is, as it name suggests, the combination of all the
other patches.

grouping.diff
This is the main change that alters the behaviour as described above.

connect.diff
Currently there is no tracking of whether items are expanded or not in
treeview. This results in erratic behaviour, e.g. tagging applies to
different images depending on cursor moves before. This connects the
treeview expanded and collapsed signals to the signal, that signifies
that anything has changed.

rename.diff
Currently the interactive rename function is executed in the iconview
context in any case. Now it is executed in the currently active view for
correct selection.

next.diff
Little correction where a change was always applied to iconview, now it
is correctly applied to whichever view is currently active.

All the changes are also available on https://github.com/imsodin/digiKam
if that is more convenient. This is related to bug 294578.

Cheers,
Simon
-------------- next part --------------
A non-text attachment was scrubbed...
Name: allchanges.diff
Type: text/x-patch
Size: 19307 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/digikam-devel/attachments/20161103/40600633/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: connect.diff
Type: text/x-patch
Size: 774 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/digikam-devel/attachments/20161103/40600633/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: grouping.diff
Type: text/x-patch
Size: 14324 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/digikam-devel/attachments/20161103/40600633/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: next.diff
Type: text/x-patch
Size: 481 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/digikam-devel/attachments/20161103/40600633/attachment-0003.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: polish.diff
Type: text/x-patch
Size: 2335 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/digikam-devel/attachments/20161103/40600633/attachment-0004.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rename.diff
Type: text/x-patch
Size: 2393 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/digikam-devel/attachments/20161103/40600633/attachment-0005.bin>


More information about the Digikam-devel mailing list