<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>366551</bug_id>
          
          <creation_ts>2016-08-09 14:34:31 +0000</creation_ts>
          <short_desc>Unclear icon-item overlay when associating people to faces</short_desc>
          <delta_ts>2022-01-31 11:02:46 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>digikam</product>
          <component>Usability-Overlay</component>
          <version>5.0.0</version>
          <rep_platform>Kubuntu</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>wishlist</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Stefano">rs4000</reporter>
          <assigned_to name="Digikam Developers">digikam-bugs-null</assigned_to>
          <cc>caulier.gilles</cc>
    
    <cc>jflemer</cc>
          
          <cf_commitlink></cf_commitlink>
          <cf_versionfixedin>7.0.0</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>21</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1612923</commentid>
    <comment_count>0</comment_count>
    <who name="Stefano">rs4000</who>
    <bug_when>2016-08-09 14:34:31 +0000</bug_when>
    <thetext>When associating faces to pictures (after the process of face detection), the dialog right now is pretty confusing:

- The name selection from list is quite OK
- There is a red cross button on the upper right corner for &quot;this is not a face&quot;
- There is a &quot;cross&quot; button next to the check button, whose action is not specified

IMVHO, there should be the following elements, one next to the other:

1. The person selection list, it would be nice if hovering the list would highlight the entry like a normal combo
2. The &quot;OK&quot; button to confirm the typed name
3. A &quot;Not-a-face&quot; button, the upper-right corner overlay is too tiny and not coherent
4. A &quot;don&apos;t care&quot; button for all those faces related to people you don&apos;t know or you don&apos;t want to tag</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1612928</commentid>
    <comment_count>1</comment_count>
    <who name="">caulier.gilles</who>
    <bug_when>2016-08-09 14:48:05 +0000</bug_when>
    <thetext>Which digiKam version do you use ?

Gilles Caulier</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1612929</commentid>
    <comment_count>2</comment_count>
      <attachid>100512</attachid>
    <who name="Stefano">rs4000</who>
    <bug_when>2016-08-09 14:48:51 +0000</bug_when>
    <thetext>Created attachment 100512
attachment-12767-0.html

Hi Gilles,

I&apos;m using 5.0.0 right now.

On Tue, Aug 9, 2016 at 4:48 PM, via KDE Bugzilla &lt;bugzilla_noreply@kde.org&gt;
wrote:

&gt; https://bugs.kde.org/show_bug.cgi?id=366551
&gt;
&gt; caulier.gilles@gmail.com changed:
&gt;
&gt;            What    |Removed                     |Added
&gt; ------------------------------------------------------------
&gt; ----------------
&gt;                  CC|                            |caulier.gilles@gmail.com
&gt;
&gt; --- Comment #1 from caulier.gilles@gmail.com ---
&gt; Which digiKam version do you use ?
&gt;
&gt; Gilles Caulier
&gt;
&gt; --
&gt; You are receiving this mail because:
&gt; You reported the bug.
&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1628815</commentid>
    <comment_count>3</comment_count>
    <who name="">jflemer</who>
    <bug_when>2016-10-03 03:45:46 +0000</bug_when>
    <thetext>I&apos;m willing to work on this issue given minimal direction from core developers.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1898086</commentid>
    <comment_count>4</comment_count>
    <who name="">caulier.gilles</who>
    <bug_when>2019-12-23 15:29:16 +0000</bug_when>
    <thetext>7.0.0-beta1 is out with new Face Recognition algorithm based on Deep
Learning/Neural Network API from OpenCV

https://download.kde.org/unstable/digikam/

Please test and give us a feedback

Thanks in advance
Gilles Caulier</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1900230</commentid>
    <comment_count>5</comment_count>
    <who name="">caulier.gilles</who>
    <bug_when>2020-01-03 14:26:07 +0000</bug_when>
    <thetext>Git commit e872abc046c8a8b7f8f51c63086d64d6b62ce4c1 by Gilles Caulier.
Committed on 03/01/2020 at 14:22.
Pushed by cgilles into branch &apos;master&apos;.

Add tootip over face name editor buttons for better usability.

M  +18   -5    core/app/items/overlays/assignnameoverlay.cpp
M  +6    -3    core/libs/widgets/itemview/itemdelegateoverlay.cpp
M  +3    -3    core/utilities/facemanagement/widgets/assignnamewidget.cpp
M  +12   -5    core/utilities/facemanagement/widgets/assignnamewidget_p.cpp
M  +10   -8    core/utilities/facemanagement/widgets/facescanwidget.cpp

https://invent.kde.org/kde/digikam/commit/e872abc046c8a8b7f8f51c63086d64d6b62ce4c1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1900234</commentid>
    <comment_count>6</comment_count>
    <who name="">caulier.gilles</who>
    <bug_when>2020-01-03 14:32:51 +0000</bug_when>
    <thetext>There is no difference between the &quot;-&quot; button on the bottom on icon view item with face overlay, and the red cross button on the top right corner. Both remove the selected faces registration to database.

Gilles Caulier</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1900235</commentid>
    <comment_count>7</comment_count>
    <who name="">caulier.gilles</who>
    <bug_when>2020-01-03 14:35:39 +0000</bug_when>
    <thetext>1/ already the case.
2/ just press Enter.
3/ 4/ both buttons do the same.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>100512</attachid>
            <date>2016-08-09 14:48:51 +0000</date>
            <delta_ts>2016-08-09 14:48:51 +0000</delta_ts>
            <desc>attachment-12767-0.html</desc>
            <filename>attachment-12767-0.html</filename>
            <type>text/html</type>
            <size>1437</size>
            <attacher name="Stefano">rs4000</attacher>
            
              <data encoding="base64">PGRpdiBkaXI9Imx0ciI+SGkgR2lsbGVzLDxkaXY+PGJyPjwvZGl2PjxkaXY+SSYjMzk7bSB1c2lu
ZyA1LjAuMCByaWdodCBub3cuPC9kaXY+PC9kaXY+PGRpdiBjbGFzcz0iZ21haWxfZXh0cmEiPjxi
cj48ZGl2IGNsYXNzPSJnbWFpbF9xdW90ZSI+T24gVHVlLCBBdWcgOSwgMjAxNiBhdCA0OjQ4IFBN
LCB2aWEgS0RFIEJ1Z3ppbGxhIDxzcGFuIGRpcj0ibHRyIj4mbHQ7PGEgaHJlZj0ibWFpbHRvOmJ1
Z3ppbGxhX25vcmVwbHlAa2RlLm9yZyIgdGFyZ2V0PSJfYmxhbmsiPmJ1Z3ppbGxhX25vcmVwbHlA
a2RlLm9yZzwvYT4mZ3Q7PC9zcGFuPiB3cm90ZTo8YnI+PGJsb2NrcXVvdGUgY2xhc3M9ImdtYWls
X3F1b3RlIiBzdHlsZT0ibWFyZ2luOjAgMCAwIC44ZXg7Ym9yZGVyLWxlZnQ6MXB4ICNjY2Mgc29s
aWQ7cGFkZGluZy1sZWZ0OjFleCI+PGEgaHJlZj0iaHR0cHM6Ly9idWdzLmtkZS5vcmcvc2hvd19i
dWcuY2dpP2lkPTM2NjU1MSIgcmVsPSJub3JlZmVycmVyIiB0YXJnZXQ9Il9ibGFuayI+aHR0cHM6
Ly9idWdzLmtkZS5vcmcvc2hvd19idWcuPHdicj5jZ2k/aWQ9MzY2NTUxPC9hPjxicj4NCjxicj4N
CjxhIGhyZWY9Im1haWx0bzpjYXVsaWVyLmdpbGxlc0BnbWFpbC5jb20iPmNhdWxpZXIuZ2lsbGVz
QGdtYWlsLmNvbTwvYT4gY2hhbmdlZDo8YnI+DQo8YnI+DQrCoCDCoCDCoCDCoCDCoCDCoFdoYXTC
oCDCoCB8UmVtb3ZlZMKgIMKgIMKgIMKgIMKgIMKgIMKgIMKgIMKgIMKgIMKgfEFkZGVkPGJyPg0K
LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tPHdicj4tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLS0tLS08d2JyPi0tLS0tLS0tLS0tLS0tLS08YnI+DQrCoCDCoCDCoCDCoCDCoCDCoCDCoCDC
oCDCoENDfMKgIMKgIMKgIMKgIMKgIMKgIMKgIMKgIMKgIMKgIMKgIMKgIMKgIMKgIHw8YSBocmVm
PSJtYWlsdG86Y2F1bGllci5naWxsZXNAZ21haWwuY29tIj5jYXVsaWVyLmdpbGxlc0BnbWFpbC5j
b208L2E+PGJyPg0KPGJyPg0KLS0tIENvbW1lbnQgIzEgZnJvbSA8YSBocmVmPSJtYWlsdG86Y2F1
bGllci5naWxsZXNAZ21haWwuY29tIj5jYXVsaWVyLmdpbGxlc0BnbWFpbC5jb208L2E+IC0tLTxi
cj4NCldoaWNoIGRpZ2lLYW0gdmVyc2lvbiBkbyB5b3UgdXNlID88YnI+DQo8YnI+DQpHaWxsZXMg
Q2F1bGllcjxicj4NCjxzcGFuIGNsYXNzPSJIT0VuWmIiPjxmb250IGNvbG9yPSIjODg4ODg4Ij48
YnI+DQotLTxicj4NCllvdSBhcmUgcmVjZWl2aW5nIHRoaXMgbWFpbCBiZWNhdXNlOjxicj4NCllv
dSByZXBvcnRlZCB0aGUgYnVnLjxicj4NCjwvZm9udD48L3NwYW4+PC9ibG9ja3F1b3RlPjwvZGl2
Pjxicj48YnIgY2xlYXI9ImFsbCI+PGRpdj48YnI+PC9kaXY+LS0gPGJyPjxkaXYgY2xhc3M9Imdt
YWlsX3NpZ25hdHVyZSIgZGF0YS1zbWFydG1haWw9ImdtYWlsX3NpZ25hdHVyZSI+U3RlZmFubzwv
ZGl2Pg0KPC9kaXY+
</data>

          </attachment>
      

    </bug>

</bugzilla>