On Mon, Feb 16, 2004 at 12:03:34PM -0500, Chris Ross wrote: > Hey there. Any feedback on this? Dominik: I see you're committing > things that aren't bug fixes, so that means you're accepting > new functionality for the post-2.6 work? Any issue with checking > in this patch?
The patch looks nice and simple, and I think I will apply it. My only concern is that it does not do any boundary checks. It should: - Check for negative and zero thickness. - Limit the thickness to what fits in a button (you do not want to allow relief thickness 25000 in a button that is 10 pixels high). > Chris Ross wrote: > > I have an FvwmIconMan inside of FvwmButtons. I was changing the > >"Frame" definition for various buttons, and wanted to change the > >relief width on the buttons in FvwmIconMan, and was suprised to find > >there was no way to do that. They're just 2. Always 2. > > > > So, I fixed this. They still default to two, but I added a > >"ReliefThickness" keyword to FvwmIconMan. Please post replies below the quoted portions original mail on this list. Ciao Dominik ^_^ ^_^ -- 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]
