[Cowbell] Re: Cowbell Patches



Hey,

> Ok, it just seemed unnecessary to have the signals both in the code and
> in glade, but since they are removed now this is ok. And yes, of course
> I meant using libglade to autoconnect the signals (i.e. replacing
> xml.BindFields() with xml.Autoconnect()).

Right.  When I first started Cowbell, I had some objectionable coding
practices ;).

> The action area border is set by Gtk. This is how it's already done in
> the save changes dialog, and it leaves a 12-pixel border between the
> edge of the window and the nearest control, which is what the HIG
> states.

Okay.  Sounds good.

> Unless I forgot something, this should be good to go, right?

Yep.  Just send me the updated patch with ChangeLog entry and I'll
commit it.

Thanks for your contribution!

-Brad

-- 
Brad Taylor
Genome Software LLC
http://www.getcoded.net

Attachment: signature.asc
Description: This is a digitally signed message part



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