[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [E-devel] E CVS: apps/e devilhorns



On Thu, 30 Nov 2006 04:03:21 -0500 Christopher Michael <cpmichael1@comcast.net>
babbled:

> Carsten Haitzler (The Rasterman) wrote:
> > On Wed, 29 Nov 2006 15:40:27 -0500 (EST) Enlightenment CVS
> > <noreply@fitx-00.ewr.caosity.org> babbled:
> > 
> > any reason NOT to have multiple? as we may change multiple windows'
> > properties? the default config dialogs want only 1 as u are editing the
> > same thing - here we need to check if there is a dialog FOR that border
> > already up - not 1 dialog total :)
> > 
> >> Enlightenment CVS committal
> >>
> >> Author  : devilhorns
> >> Project : e17
> >> Module  : apps/e
> >>
> >> Dir     : e17/apps/e/src/bin
> >>
> >>
> >> Modified Files:
> >> 	e_int_config_borders.c 
> >>
> >>
> >> Log Message:
> >> Use the config dialog find functions to avoid having 2 border dialogs open
> >> for the same window.
> <--- snip --->
> 
> After sleeping on it, I totally agree :) I'll look into 
> modifying/creating config dialog search functions. Basically, I believe 
> we need something that will search the config dialog collection for a 
> given class (which we do have already) and return that config dialog so 
> we can compare cfd->border to the current one being edited. Will look 
> into today :)

well you can have bd->border_dialog that does just that for you - explicitly -
if bd->border_dialog != NULL then raise/show that dialog - otherwise, create
new (and on delete - set bd->... to NULL)

> P.S. - Happy Birthday Raster !! :) (belated tho it may be) Hope you had 
> a good one.

thanks man... another foot into the old mens club for me

-- 
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler)    raster@rasterman.com
裸好多
Tokyo, Japan (東京 日本)