[Digikam-devel] [Bug 209189] Make albumview rotation buttons optional

[Marcel Wiesweg]

https://bugs.kde.org/show_bug.cgi?id=209189


Marcel Wiesweg <marcel.wiesweg at gmx.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |marcel.wiesweg at gmx.de




--- Comment #10 from Marcel Wiesweg <marcel wiesweg gmx de>  2009-10-03 17:27:33 ---

> > IMHO there is no need for a secondary selection method, there is a standard way
> > of doing this in all KDE programs (hold the CTRL-key and then left click).. 

With Dolphin and KDE4, the (+) icon has become a standard with KDE, a lot of
thought and discussion about usability has went into this.

> 
> Gilles, what do you think? The overlay for the selection is added by the base
> class ImageCategorizedView. I don't know if it's somewhere else used, so the
> only non-intrusive way to remove that overlay in the view - that I see - is
> adding a method to remove the overlay to ImageCategorizedView. But that sounds
> like duplicate work... first adding it in the constructor and then removing it
> afterwards. Maybe we should add a boolean parameter to the constructor to
> auto-add this overlay defaulting to true for backwards-compatibility.
> AlbumIconView can set this to false and use addSelectionOverlay based on the
> settings.

Overlays can be added, removed, and re-added again. (never tested, but should
be). It's added in the constructor just for convenience. Store the pointer in
the private class and just add or remove it.


> > >Regarding the space: There is not space reserved specially for these overlays.

Intentionally

> > Regarding the border, I don't understand why the border is painted whenever I
> > touch an image by passing the mouse over it but it (the border) isn't removed
> > after the mouse left the active area - scrolling the list does remove the
> > border :-(...

which border is this?

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.