GNOME Bugzilla – Bug 151275
VariablesMap additions
Last modified: 2011-01-16 23:33:40 UTC
This patch allows to: *to easy connect buttons and menuitems to an handler. *Glade::VariablesMap handles Entry, ComboTextEntry, SpinButton, *Scale, Calendar and CheckBox. *Improved the example to handle new VariablesMap types.
Created attachment 31045 [details] [review] connection_variablesmap.patch
This looks good, but we must wait for GNOME 2.9 to apply it, because GNOME 2.7 is past API-freeze. Thanks. In future please, - do not change the version number in your patch. - try to use pVar->get_day() rather than (*pVar).get_day(). - do not close your bugs as patched immediately after creating them.
Committed to HEAD. I changed connect() to connect_clicked(), to avoid confusion with xml_glade_connect(). I also corrected some english and made the coding style consistent. Thanks a lot. This works really well.