On 13 Mar 2003 22:43:22 +0100, [EMAIL PROTECTED] wrote:
> 
> On Thu, Mar 13, 2003 at 07:50:51PM +0100, Remko Troncon wrote:
> > Hello FVWM workers,
> > 
> > Enclosed is a patch i proposed a few weeks ago, but which wasn't accepted at
> > the time because of another bug in FvwmButtons. So, i am reposting it for
> > revision, and would like it committed. The patch adds a new option to 
> > FvwmButtons 'Relief <reliefmode>' (with <reliefmode> one of Up, Down, Both),
> > specifying in which state of the button a relief should be drawn.

I hoped you will redo this using UpRelief and DownRelief options... :)

> > Once this is done, I would like to implement two new options for Panel in 
> > FvwmButtons.
> > The first one is 'OpenCreates': When a button controlling a panel is
> > clicked, and the panel does not exist yet, the command of the Panel
> > is executed to create the panel.
> > This new option induces another possible option, 'NoStart': When 
> > FvwmButtons 
> > is started or restarted, the 'command' of the panel is _not_ executed. 
> > 
> > I personally have a need for both options, both for a respawning and a non-
> > respawning panel. Does anyone have any comments on this ? 
> > The names of the options might not be very good, so suggestions on those 
> > are also very welcome.

I may suggest to do it a bit differently, new options for Panel: Respawn
and SpawnOnOpen. This should achieve the functionality you want, but in a
bit different (and I hope a more intuitive) way,

> Because we are in feature freeze, no new features are accepted at
> the whim of one developer.  However, I agree to apply this patch
> if a) someone else proof reads and applies it, and b) my fellow
> developers agree.  I thought the patch already went into the code.

I want to have the functionality in the patch, but the way it is done now
it is not very extendible and as it is now it should be named
"DrawReliefOn <state>", not "Relief <reliefmode>".

Ideally I would like to have more relief styles in the future, like:

  Relief <state> <style>   # state = up/down/both, style=clean/shadow/edge

Or to use "UpRelief/DownRelief <state>" syntax for this.
Additionaly I would like not to encourage a special meaning of negative
Frame value and do it like FvwmIconMan does it. I.e. Frame is taken only
to be positive and we have style=raised/sunk/raisededge/sunkedge values.
By default style for "up" is "raised" and for down is "sunk", like now.

So although I want to have this functionality, I can't vote "yes" yet
for the patch. This needs more discussion so we don't need to change the
syntax in the future.

Regards,
Mikhael.
--
Visit the official FVWM web page at <URL:http://www.fvwm.org/>.
To unsubscribe from the list, send "unsubscribe fvwm-workers" in the
body of a message to [EMAIL PROTECTED]
To report problems, send mail to [EMAIL PROTECTED]

Reply via email to