Re: Automatic signal connection



On Wed, Jan 21, 2009 at 6:40 PM, Kai Willadsen <kai willadsen gmail com> wrote:
> 2009/1/19 Stephen Kennedy <stevek gnome org>:
>> Good point. The method now handles the case where "widget" is a list.
>> For the on/after conflict, how about changing the uimanager handler
>> names to action_xxx or ui_xxx?
>
> Changing them to action_xxx sounds reasonable. I can do this in the
> UIManager branch if you like... I was hoping to land that sometime
> this weekend if you're okay with the changes as described (I'll try to
> get around to screenshotting some of this too). It's dependent on
> getting the toolbar dropdowns working, but I've made some progress on
> that, and also added toggle menu items for filters to the View menu. I
> think it works pretty well from a UI point of view, though automatic
> updating of comparisons in FileDiff needs some work to
> do-the-right-thing.
>
> Auto-attaching to top-level widgets is going to require a lot of
> renaming of methods, but probably nothing drastic. Also, the current
> signal-name extraction looks like it won't work for hyphenated signals
> (e.g., button-press-event). I assume that just replacing dashes with
> underscores should work.

> (P.S. did you mean this to go to the list as well?)

Yes, I accidentally dropped the cc. I have quoted above for the record.

The UiManager changes sound great. If you've not already renamed the
methods, there's no rush as they can be renamed later as components
are migrated to automatic connection.

I hope the migration can be smooth - for a little while at least, both
systems can operate together. BTW underscores in signal names work as well
as dashes - haven't seen this documented but it's been that way for as
long as I can remember.

Stephen.


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