On ne 1. března 2009, Ruben Stein wrote:
> Thanks Laurent for your instant adjustments.
>
> From: Vladimir Nadvornik - 2009-03-01 11:03
>
> > - editors.c - Geeqie now uses desktop files and it is not possible to add
> > non-standard parameters directly on commandline. It should be possible to
> > export the coordinates in environment variables however.
> > BTW, which editors can use this?
>
> I see, the simple and fast custom commands are one feature I love at
> gqview. ;) I use it for scripts and home grown applications. So I can
> use GQView for watching the pictures my algorithms produced, and start
> own analysis tools using the custom commands. Do you plan to support
> custom commands in the future again?

The single-line commandline from GQView has several problems:
- it is hard to set distribution defaults
- it uses non-standard macros
- it is hard to edit more complicated commands
- it is hard to contribute the good commands upstream

Desktop files fixes these problems and adding new applications is not
much harder, just edit a template desktop file and copy it to 
~/.geeqie/applications. When I think about it, it would be easy
to add the "copy and edit template" function directly to Geeqie.


> Maybe I will find the time to read through the destop-files approach
> and try to do it with this, but for my purposes of prototyped scripts
> the simple commandline from gqview is easier to use.
>
> > - layout.c - IMO the menu is not needed.
>
> Thought it would raise the willingness to integrate the feature. After
> all it flashes permanently.
>
The flashing really does not look very good. Maybe use fixed-size formatting?
Another possibility would be to add it to sidebar. It would have more room
there and the enabling and disabling would work automatically.


> > - pixbuf-renderer.c - does it have to store everything? IMO mouse
> > coordinates should be enough and the pixel coordinates and RGB can be
> > computed on demand.
>
> I changed it in this patch so rgb-data is only calculated in the
> getter. Anyway - I would hold the pixel coordinates. I would do so
> because there is no need to call the layout_status_update_pixel if the
> mouse points still to the outside of the buffer.

So store only pixel coordinates then? Having both means extra care to
keep them synchronized.

BTW, hardcoded layout_status_update_pixel call is not very good. IMO correct 
way would be to use a signal (grep for "scroll_notify" to see how to 
implement this): layout will register a callback when it wants to be informed 
about pixel changes and from the callback queries the pixbuf-renderer for 
details.

> But to know this, you 
> first have to calculate the pixel data. So they are computed anyway
> before calling the statusbar-update and calculating them a second time
> from the layout_status_update_pixel would not make sense imho.
>
> I also changed sth in the zoom-function so the pixelinfo gets updated
> if the picture is changed e.g. by scrollwheel. Additionally there was
> an off-by-1-fault in the color-update guard.
>

Vladimir


------------------------------------------------------------------------------
Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA
-OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise
-Strategies to boost innovation and cut costs with open source participation
-Receive a $600 discount off the registration fee with the source code: SFAD
http://p.sf.net/sfu/XcvMzF8H
_______________________________________________
Geeqie-devel mailing list
Geeqie-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geeqie-devel

Reply via email to