<div dir="ltr">Hi Simon,<div><br></div><div>Thanks for your patches. It's very appreciate.</div><div><br></div><div>Please, in the future, do not send patches and bugfixes directly in mailing list. Use bugzilla instead. Attachments size in ML is limited. </div><div><br></div><div>I created one with all patches :</div><div><br></div><div><a href="https://bugs.kde.org/show_bug.cgi?id=372027">https://bugs.kde.org/show_bug.cgi?id=372027</a><br></div><div><br></div><div>Best</div><div><br></div><div>Gilles Caulier</div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-11-03 13:00 GMT+01:00 Simon Frei <span dir="ltr"><<a href="mailto:freisim93@gmail.com" target="_blank">freisim93@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Currently when a grouped image is selected all operations (tagging,<br>
renaming, ...) are only applied to the head of the group. This is not<br>
consistent with the options to display items: In iconview, there is<br>
grouped and ungrouped, in tableview ungrouped, grouped and tree. So I<br>
have control over whether I want to display images individually or as<br>
groups. However this control does not extend to actions.<br>
<br>
The attached patches change the behaviour:<br>
When images are grouped or not expanded in treeview, actions apply to<br>
the selected images and the grouped images. In Ungrouped view or when<br>
images are expanded in treeview, actions apply only to the selected images.<br>
<br>
In the process I uncovered some more or less related issues, that is why<br>
I have split the patch up. Every single one is functional.<br>
"allchanges.diff" is, as it name suggests, the combination of all the<br>
other patches.<br>
<br>
grouping.diff<br>
This is the main change that alters the behaviour as described above.<br>
<br>
connect.diff<br>
Currently there is no tracking of whether items are expanded or not in<br>
treeview. This results in erratic behaviour, e.g. tagging applies to<br>
different images depending on cursor moves before. This connects the<br>
treeview expanded and collapsed signals to the signal, that signifies<br>
that anything has changed.<br>
<br>
rename.diff<br>
Currently the interactive rename function is executed in the iconview<br>
context in any case. Now it is executed in the currently active view for<br>
correct selection.<br>
<br>
next.diff<br>
Little correction where a change was always applied to iconview, now it<br>
is correctly applied to whichever view is currently active.<br>
<br>
All the changes are also available on <a href="https://github.com/imsodin/digiKam" rel="noreferrer" target="_blank">https://github.com/imsodin/<wbr>digiKam</a><br>
if that is more convenient. This is related to bug 294578.<br>
<br>
Cheers,<br>
Simon<br>
</blockquote></div><br></div>