Re: [Gnustep-cvs] r24671 - /libs/gui/trunk/Source/NSAlert.m

[Sergii Stoian]

Hi, Fred.

I think the setRefusesFirstResponder is better because user should click 
with mouse on non-default buttons. While default button used for 
non-destructive procedure (as to NextStep User Interface Guilde) user can't
select other buttons by accident (read using Tab button).

As for button style: agreed. I'll make changes to svn.

On 2/21/07, Fred Kiefer <address@hidden> wrote:

I am not sure, but wouldn't it be better to just switch of the display
of the first responder frame by calling setShowsFirstResponder: on the
cell with NO as the argument?

On a not so related issue, you had to set the highlight behaviour for
these buttons explicitly. What about setting the type of all these
buttons to be NSMomentaryPushInButton in _makeButtonWithRect:. This
looks more consistent to me. The default button would still be different
though.

Sergii Stoian schrieb:
> Author: stoyan
> Date: Wed Feb 21 01:46:01 2007
> New Revision: 24671
>
> URL: http://svn.gna.org/viewcvs/gnustep?rev=24671&view=rev
> Log:
> * Source/NSAlert.m: (-_initWithoutGModel):
> Make buttons refuse first responder to avoid drawing
> dashed frame.
>
> Modified:
>     libs/gui/trunk/Source/NSAlert.m
>

-- 
Sergii Stoian, ProjectCenter maintainer