[Digikam-devel] Digikam menu usability improvements

[Caulier Gilles]

Le Mardi 23 Janvier 2007 22:10, Luka Renko a écrit :
> Hello,
>
> I would like to implement some usability improvements for digiKam and
> friends. I have started with some "low hanging fruit" and have attach two
> simple patches for DigiKam Editor and ShowFoto to have View menu on more
> appropriate location. See bug [1] for attached patches. I would like if one
> of the developers would review them, then I can commit them into KDE svn.
>
> I would also like to move "Full Screen Mode" to View menu (currently in
> settings) in all three applications and rename the option in DigiKam
> (currently named "Toggle Full Screen Mode" in Digikam). This would also
> make it consistent with Konqueror which also has this in View menu.
>
> Similar one would also be moving Slideshow to View menu - see [3]. I would
> really like to here if there are any objections to change this.
>
> Next, larger one is DigiKam menu structure, which I think is more
> complicated than other apps (Editor, ShowFoto) and needs a bit more
> thought. See [4].
>
> Regards,
> Luka
>
> [1] http://bugs.kde.org/show_bug.cgi?id=140320
> [2] http://bugs.kde.org/show_bug.cgi?id=127617
> [3] http://bugs.kde.org/show_bug.cgi?id=140303
> [4] http://bugs.kde.org/show_bug.cgi?id=89365

Luka,

There is also another point about menu in editor to improve : the 
View/Histogram option...

This one have been added with digikam 0.7.x series by a contributor. With 0.7 
and 0.8 series, there is no sidebar available. This is why the patch have 
been accepted.

Since 0.9.0, the histogram is available in Color sidebar tab. I think than 
View/Histogram option are now redondant. Also, the View/Histogram options are 
uncomplete : you cannot change the scale for example...

Personnally, i would to remove this option from Image Editor and Showfoto. By 
this way, the B.K.O file # 110514 can be closed (obsolete)

Of course, this must be done only if users are agree with this changes. It 
always easy to remove code than to add (:=)))

The source code to fix is :

digikam/utilities/imageeditor/canvas/canvas.cpp
digikam/utilities/imageeditor/canvas/dimginterface.cpp
digikam/utilities/imageeditor/editor/editorwindow.cpp

I'm not sure if we can do it before 0.9.1.

Please give me your viewpoints. Thanks in advance

Gilles