https://bugs.kde.org/show_bug.cgi?id=398376
daniel-other+kde...@dadosch.de changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |daniel-other+kdebug@dadosch
| |.de
--- Comment #4 from daniel-other+kde...@dadosch.de ---
(In reply to caulier.gilles from comment #3)
> Git commit 05b9a534d89234b7571d5dade8067c490ab896cf by Gilles Caulier.
> Committed on 26/12/2019 at 17:30.
> Pushed by cgilles into branch 'master'.
>
> Faces Workflow simplication and improvements: remove face scan dialog and
> embed Face scan settings in People left sidebar.
> disable settings view while a face job is running.
> Catch face detector tool complete/cancelled signal to re-enable settings
> view.
> By this way, this will prevent to run 2 concurrent face jobs at the same
> time.
> Related: bug 412168, bug 380336
>
> M +45 -31 core/app/views/sidebar/leftsidebarwidgets.cpp
> M +1 -0 core/app/views/sidebar/leftsidebarwidgets.h
> M +1 -1 core/utilities/facemanagement/CMakeLists.txt
> R +15 -45 core/utilities/facemanagement/widgets/facescanwidget.cpp
> [from: core/utilities/facemanagement/widgets/facescandialog.cpp - 089%
> similarity]
> R +9 -11 core/utilities/facemanagement/widgets/facescanwidget.h [from:
> core/utilities/facemanagement/widgets/facescandialog.h - 079% similarity]
> R +8 -11 core/utilities/facemanagement/widgets/facescanwidget_p.h
> [from: core/utilities/facemanagement/widgets/facescandialog_p.h - 090%
> similarity]
> M +1 -1 core/utilities/maintenance/facesdetector.cpp
> M +3 -3 core/utilities/maintenance/facesdetector.h
> M +1 -0 core/utilities/maintenance/maintenancetool.cpp
> M +11 -3 core/utilities/maintenance/maintenancetool.h
>
> https://invent.kde.org/kde/digikam/commit/
> 05b9a534d89234b7571d5dade8067c490ab896cf
With the Face scan settings in People left sidebar, it is harder to move tags
in the upper half into a different category, especially on lower resolution
screen. (It only shows 5 entries on the tag tree for me). One solution might be
to make face scan setting widget collabsable and only show the main scan
button.
--
You are receiving this mail because:
You are watching all bug changes.
