Bug 426699

Summary: "Ignored" faces category should not be positioned alphabetically, but on top of the list
Product: [Applications] digikam Reporter: MarcP <iwannaberich>
Component: Faces-WorkflowAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: iwannaberich, metzpinguin
Priority: NOR    
Version: 7.2.0   
Target Milestone: ---   
Platform: Appimage   
OS: Linux   
Latest Commit: Version Fixed In: 7.2.0
Sentry Crash Report:

Description MarcP 2020-09-18 19:05:06 UTC
SUMMARY

I have been playing with the 7.2 beta, and I absolutely love the changed made during the GSoC2020. The whole face recognition is perfectly usable now. This year's student has done a brilliant job.

Just one little thing. Like "Unconfirmed" and "Unknown" are always placed on top of the faces list, so they are always visible and available, the new "Ignored" category should also be there on top. Currently, it is positioned alphabetically under the "I" letter, so in my case I have to scroll approximately 200 faces until I find the "Ignored" category.

It's a minor thing, but I believe could be easily fixed.
Comment 1 Maik Qualmann 2020-09-18 19:24:20 UTC
The "Ignored" tag should appear at the very end of the list, this is the student's intention. People of the faces that have not yet been confirmed are also pushed up. Only when there are no more unconfirmed ones do they line up alphabetically.

Maik
Comment 2 MarcP 2020-09-18 19:28:16 UTC
Oh, ok, So this was done on purpose, I thought it appeared in the middle of the list, sorry, my fault. 

But I still think it's not very visible, as I had to manually search for it. I believe interface elements should not be that "hidden".

Anyway, this report is irrelevant now, you can close it if you want.
Comment 3 Maik Qualmann 2020-09-18 19:37:39 UTC
Another question, is "Ignored" really at the end of the list for you, or are there still names follow?

Another thing is the sorting of the face categories when sub tags are activated and the people tag is selected. Since I had to fix the view, I decided to move the unknown and ignored faces to the end. So that the confirmation can be processed from above without scrolling through all unknown faces. Good or bad?

Maik
Comment 4 MarcP 2020-09-18 19:43:19 UTC
Yes, I confirm that "Ignored" appears at the end of the list, and "Unconfirmed" and "Unknown" at the beginning.

Mmm, in my case, I would prefer them to be always visible, even if I scrolled, because I consider them part of the interface, not faces. This is why I suggested to be on top of the list. I would even show them in another font or color. I fear that if they are at the end, the user might never see those elements (as it happened to me, I had to search "Ignored" because I knew it had to be there, even if I couldn't find it).
Comment 5 Maik Qualmann 2020-09-19 04:40:09 UTC
Git commit aa4f7ce6522c1c9559619c2dd3af5207a8a451ac by Maik Qualmann.
Committed on 19/09/2020 at 04:39.
Pushed by mqualmann into branch 'master'.

move the "Ignored" tag to the top of the list
FIXED.IN: 7.2.0

M  +2    -4    NEWS
M  +3    -3    core/libs/models/albumfiltermodel.cpp

https://invent.kde.org/graphics/digikam/commit/aa4f7ce6522c1c9559619c2dd3af5207a8a451ac