java-gnome-hackers

[java-gnome-hackers] Coverage of GtkSwitch

[Guillaume M. <res...@gm...>]

Hi,

Here is I propose another branch that I'd like to see merged for the
next java-gnome release. The branch is located at:

    hackers/guillaume/gtk-switch

From the branch location you probably understand that it covers the
GtkSwitch widget that was introduce with GTK+ 3. That's a pretty simple
branch, I just had to play with some C functions to be able to emit and
catch a signal when the user plays with the Switch.

Any feedbacks are welcome (as usual).

Cheers,

--
Guillaume Mazoyer

Re: [java-gnome-hackers] Coverage of GtkSwitch

[Andrew C. <an...@op...>]

On Tue, 2011-10-25 at 01:38 +0200, Guillaume Mazoyer wrote:

> From the branch location you probably understand that it covers the
> GtkSwitch widget that was introduce with GTK+ 3. That's a pretty simple
> branch, I just had to play with some C functions to be able to emit and
> catch a signal when the user plays with the Switch.

The other 'notify::property' signals we have exposed have been as
Widget.NotifyProperty, for example

http://java-gnome.sourceforge.net/doc/api/4.1/org/gnome/gtk/TextBuffer.NotifyCursorPosition.html

I'm not saying that this is the best solution, but we should probably be
consistent. If so, then maybe you could change it to
Switch.NotifyActivated ? 

[Yes, I agree it's more cumbersome, but it would be more consistent,
maybe. What do you think?]

AfC
Prague

Re: [java-gnome-hackers] Coverage of GtkSwitch

[Guillaume M. <res...@gm...>]

On Thu, Oct 27, 2011 at 11:19:26AM +0200, Andrew Cowie wrote:
> I'm not saying that this is the best solution, but we should probably be
> consistent. If so, then maybe you could change it to
> Switch.NotifyActivated ? 
Yes I think that it's more consistent.
I changed the signal name and pushed it into the branch.

--
Guillaume Mazoyer

Re: [java-gnome-hackers] Coverage of GtkSwitch

[Andrew C. <an...@op...>]

On Thu, 2011-10-27 at 20:13 +0200, Guillaume Mazoyer wrote:
> Yes I think that it's more consistent.

Ok

> I changed the signal name and pushed it into the branch.

Thanks. Merged to 'mainline'

AfC
Prague