[KPhotoAlbum] Multi-image annotation dialog annoyance

[Robert Krawitz]

On Sun, 20 Nov 2016 21:23:02 +0100, Johannes Zarl-Zierl wrote:
> Hi Robert,
>
>
> On Sonntag, 20. November 2016 14:30:09 CET Robert Krawitz wrote:
>> So a bit more information.
>> 
>> It isn't flashing each image in turn; it's flashing just one image.
>> 
>> Also, there are two buttons at the top right of each widget.  The one
>> on the extreme right is a close button; the other one I can't figure
>> out, but if I click on it, the flickering stops and it displays
>> normally.
>
> Ok, that makes a difference - that is a known (albeit nasty) bug
> introduced during porting to Qt5. We had thought we had fixed it,
> but apparently it can still be triggered sometimes.  I'll take
> another look at this regression. In contrast to the multi-image
> flickering I have some idea where this problem comes from...

OK.

>> But I have a few other complaints about the annotation dialog:
>> 
>> 1) If I close one of the widgets (or turn it off in the
>>    Options/Settings dialog), it goes away, but the next time I enter
>>    the annotation dialog, all of the default widgets come back
>
> You need to save the window layout to make it permanent. In the
> annotation dialog, select "Options…|Configure window layout…|Save
> current window setup".

That's awfully unintuitive (especially since you have to go through
Configure window layout..." once for each change you want to make.  It
would make more sense as an actual window (not just a menu) that would
let you make your selections and then click save (or apply/save/cancel).

>> 2) In the image preview window, there's a button "Train face
>>    recognition database automatically".  It's grayed out, so I can't
>>    turn it off.  I don't care about face recognition, so I don't want
>>    it at all.
>
> You should not get that button if you compile without libkface. If
> you really don't want it, I think it's best to remove
> libkf5kface-dev (on debian-style systems; probably libkface-devel on
> rpm distros).

This should be a runtime option, not a compile time option.
Distributions are going to compile it with libkface; it should be
possible to disable it.

>>    (the positionable tags and face
>>    recognition aren't usable at all to me even in single image mode
>>    since I haven't configured the requisite features in my options).
>
> To use positionable tags, you need to mark one or more categories as
> positionable in the categories tab in the settings dialog. Btw. you
> can use positionable tags even if you don't have face
> detection/recognition.

I'm not interested in using positionable tags at all, actually.

>>    Also, to the right of the next/previous buttons below the image,
>>    there are 5 other buttons (rotate left, right, etc) which are
>>    grayed out in multi-image mode
>>    Since they're completely useless in multi-image mode, should they
>>    even be there at all?
>
> Short answer: yes.
>
> While it would be technically possible to remove these buttons in
> multi-image and search mode, the annotation dialog is already kind
> of messy and that would definitely make matters worse. Besides, we
> have had this situation basically forever, even more so if you take
> a look at the search dialog.

Well, whatever -- except that there's nothing I can do in the actual
dialog to enable them, so they're a distraction.
-- 
Robert Krawitz                                     <rlk at alum.mit.edu>

***  MIT Engineers   A Proud Tradition   http://mitathletics.com  ***
Member of the League for Programming Freedom  --  http://ProgFree.org
Project lead for Gutenprint   --    http://gimp-print.sourceforge.net

"Linux doesn't dictate how I work, I dictate how Linux works."
--Eric Crampton