Re: [Gqview-devel] Diplay pixel coordinates, and rgb-data / pixel coordinates for editor

[Vladimir N. <nad...@su...>]

Hi Ruben,

On ne 1. března 2009, Ruben Stein wrote:
> I missed a feature in gqview, so I implemented it today. It is an
> optional status-bar add-on showing the pixel-coordinates of the mouse
> and the rgb-values of this pixel. I also added %x and %y as
> editor-macros working together with %p to use the pixel values for a
> call to a script. Furthermore I added a checkbox in the view-menu to
> turn off the feature as it may be disturbing to see the permanently
> changing values if you do not need them.
>
> Referring to former feature requests I think maybe someone might want
> to use it:
> https://sourceforge.net/tracker/index.php?func=detail&aid=1145036&group_id=
>4050&atid=354050
> https://sourceforge.net/mailarchive/message.php?msg_name=40F9E2BE.9080606%4
>0techspecs.com
>
> It was my first try adding _senseful_ ;) features to an open source
> project, so any advices are appreciated. Unfortunately I did not
> recogise that gqview is outdated, so I learned my first lesson. Anyway
> ... the attached patch applies to GQView 2.1.5 src-dir, diffed against
> the svn-tag in the geeqie-repository of that version.
>

This seems to be a useful feature and I'd like to add it to Geeqie. However
there are some issues with porting the patch:

- 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?

- layout.c - IMO the menu is not needed.

- 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.


Please write on geeqie-devel@ if you want to port the patch.

Thanks,
Vladimir