Bug 405327

Summary:	Position and size of faces display depends on configuration setting
Product:	[Applications] digikam	Reporter:	Hanspeter Halle <hphalle>
Component:	Faces-Workflow	Assignee:	Digikam Developers <digikam-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	hphalle
Priority:	NOR
Version First Reported In:	6.0.0
Target Milestone:	---
Platform:	Microsoft Windows
OS:	Microsoft Windows
Latest Commit:	https://commits.kde.org/digikam/c292bbb18e08dc6a1983613b8fd22a7cbafb465e	Version Fixed/Implemented In:	6.1.0
Sentry Crash Report:
Attachments:	Screenshot with check box "Preview is zoomed to the original image size" selected Screenshot with check box "Preview is zoomed to the original image size" not selected

Description Hanspeter Halle 2019-03-10 19:10:23 UTC

The place a face tag displayed depends on the setting of "Preview is zoomed to the original image size". If this parameter is changed while face tags are displayed the tags move around and even outside the image size.


STEPS TO REPRODUCE
1. Open an image with face tags displayed
2. Change setting->preview->Preview is zoomed to the original image size

OBSERVED RESULT
Face tag display  moves to another position of the image

EXPECTED RESULT
Face tag stays at the same position

SOFTWARE/OS VERSIONS
Windows: 10
ADDITIONAL INFORMATION
The laptop used has a HD-Display

Comment 1 Hanspeter Halle 2019-03-10 19:20:16 UTC

Created attachment 118695 [details]
Screenshot with check box "Preview is zoomed to the original image size" selected

Comment 2 Hanspeter Halle 2019-03-10 19:21:29 UTC

Created attachment 118696 [details]
Screenshot with check box "Preview is zoomed to the original image size" not selected

Comment 3 Maik Qualmann 2019-03-12 19:44:29 UTC

Git commit c292bbb18e08dc6a1983613b8fd22a7cbafb465e by Maik Qualmann.
Committed on 12/03/2019 at 19:42.
Pushed by mqualmann into branch 'master'.

remove the option zoom to original size
It is not good for the position and size of face regions
if this option is disabled and the system no longer knows
the original size of the image. For the zoom to 100% of a
reduced preview, we find another solution.
FIXED-IN: 6.1.0

M  +2    -1    NEWS
M  +0    -2    core/libs/settings/applicationsettings.cpp
M  +0    -1    core/libs/settings/applicationsettings_p.cpp
M  +0    -1    core/libs/settings/applicationsettings_p.h
M  +0    -2    core/libs/threadimageio/preview/previewsettings.cpp
M  +0    -1    core/libs/threadimageio/preview/previewsettings.h
M  +1    -9    core/libs/threadimageio/preview/previewtask.cpp
M  +2    -10   core/utilities/setup/album/setupalbumview.cpp

https://commits.kde.org/digikam/c292bbb18e08dc6a1983613b8fd22a7cbafb465e