Re: More about the feature list

Carles Pina i Estany a écrit :
>> - Red eye correction: This one is new, but I keep stumbling on pictures 
>> which need this kind of fix, so I figured I'll implement it
> 
> I have some reticence about editing images in gwenview (I afraid that
> adding too much things convert gwenview in a image editor). Ok, I like
> the idea of Red Eye correction but I don't know where to "put" the
> limit.
> Maybe this would be better as a KIPI plugin? But then I think that is
> not so integrated...

Problem with KIPI plugins is that they work on URLs, not on the actual 
image data. As you said, it wouldn't be as integrated as implementing it 
in Gwenview.

> 
>> - Support for tagging with Nepomuk
> 
> I'm not using it, so I don't know (yet) about Nepokum. Doing a search
> ifor "neopomuk" (misstyping) in Google, fifth entry is:
> http://mail.kde.org/pipermail/digikam-devel/2007-June/013151.html
> (I have not read it, but the title is about Digikam + Neopomuk)

Interesting. Don't worry about not using Nepomuk, I think nobody does 
for now :-), it's KDE4 future. I like this project because I think 
having a system-wide solution to store meta data is the only good way to 
work with meta data.

> Some weeks ago I tried to add the "-s" switch (open in slideshow mode,
> and a parameter for the seconds) and -f (full screen).
> I think that I was not using the correct aproach. Aurelien, could you
> tell me in 5 lines how would you do it? I had problems because I was
> calling the toggleFullScreen or toggleSlideshow from some "random"
> places where I was guessing that everything was load (in src/main.cpp?
> or src/gwenview.cpp, don't know now). But I didn't like... 

I would not call the toggle* methods directly: it's better to trigger 
the associated actions with code like mFullScreenAction->trigger(). This 
way you make sure the "checked" property of the action is correct.

I just gave it a try and was able to start Gwenview in fullscreen mode 
by adding this line (and a few #includes) in main.cpp:

   window->actionCollection()->action("fullscreen")->trigger();

Just after window->show()

Slideshow may be a bit more difficult, I haven't investigated yet.

Aurélien

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/