Re: [Gimp-developer] Extending GPConfig

2004-09-09 Thread Sven Neumann 
Hi,

here's another preview update for you. A while ago this has been
written here:

> (1) Add a popup menu (to GimpPreview) that allows to configure the
> checkerboard per preview instance.

This has been done in the meantime. The menu API might need some
changes to allow for these menu items to be merged with other stuff
that might be useful for previews.

Other things that have happened with respect to previews:

 - previews now respect the selection
 - previews resize with the plug-in dialog
 - previews now have a navigation popup window like the one we have in
   the image window

> (2) Add another GimpPreview derived widget to libgimpui. That widget
> would be used from plug-ins such as the Plasma plug-in which don't
> need the GimpDrawablePreview functionality. The widget could be
> called GimpSimplePreview (an idea for a better name would be very
> much appreciated). It would embed a GimpPreviewArea in a sunken
> GtkFrame and it would take care of setting a proper size as well
> as configuring the preview area to the gimprc values.

This hasn't been done yet. Probably because it is still looking for a
reasonable name. Everything else would be trivial.


Sven
___
Gimp-developer mailing list
[EMAIL PROTECTED]
http://lists.xcf.berkeley.edu/mailman/listinfo/gimp-developer

Re: [Gimp-developer] Extending GPConfig

2004-09-03 Thread Sven Neumann 
Hi,

just FYI: we've by now done the discussed changes to the GPConfig
struct. GimpPreviewArea has been changed to make the checkerboard
pattern configurable and GimpDrawablePreview already passes the config
values to the preview area. What remains to be done is:

(1) Add a popup menu (to GimpPreview) that allows to configure the
checkerboard per preview instance.

(2) Add another GimpPreview derived widget to libgimpui. That widget
would be used from plug-ins such as the Plasma plug-in which don't
need the GimpDrawablePreview functionality. The widget could be
called GimpSimplePreview (an idea for a better name would be very
much appreciated). It would embed a GimpPreviewArea in a sunken
GtkFrame and it would take care of setting a proper size as well
as configuring the preview area to the gimprc values.


Sven
___
Gimp-developer mailing list
[EMAIL PROTECTED]
http://lists.xcf.berkeley.edu/mailman/listinfo/gimp-developer

Re: [Gimp-developer] Extending GPConfig (was: Should the checkerboard be linked to the window or to the image?)

2004-09-02 Thread Manish Singh 
On Thu, Sep 02, 2004 at 08:08:23PM +0200, Michael Natterer wrote:
> struct _GPConfig
> {
>   guint32  version;
>   guint32  tile_width;
>   guint32  tile_height;
>   gint32   shm_ID;
>   gdouble  gamma;
>   gint8install_cmap;
>   gint8show_tool_tips;
>   gint32   min_colors;
>   gint32   gdisp_ID;
>   gchar   *app_name;
>   gchar   *wm_class;
>   gchar   *display_name;
>   gint32   monitor_number;
> };
> 
> I see two options: 
> 
> - using the place "gdouble gamma" uses.
> - using the two unused bytes after "gint8 show_tool_tips".
> 
> I'd suggest we do both and change the struct to:
> 
> struct _GPConfig
> {
>   guint32  version;
>   guint32  tile_width;
>   guint32  tile_height;
>   gint32   shm_ID;
>   gint8gimp_reserved_1;
>   gint8gimp_reserved_2;
>   gint8gimp_reserved_3;
>   gint8gimp_reserved_4;
>   gint8gimp_reserved_5;
>   gint8gimp_reserved_6;
>   gint8gimp_reserved_7;
>   gint8gimp_reserved_8;
>   gint8install_cmap;
>   gint8show_tool_tips;
>   gint8gimp_reserved_9;
>   gint8gimp_reserved_10;
>   gint32   min_colors;
>   gint32   gdisp_ID;
>   gchar   *app_name;
>   gchar   *wm_class;
>   gchar   *display_name;
>   gint32   monitor_number;
> };
> 
> Which should be binary compatible on all platforms.

Sounds good, and gimp_gamma should return 1.0 always (and be marked as
deprecated).

-Yosh
___
Gimp-developer mailing list
[EMAIL PROTECTED]
http://lists.xcf.berkeley.edu/mailman/listinfo/gimp-developer