[Gossip-dev] final account rework
Xavier Claessens
xclaesse at gmail.com
Sun Oct 8 00:23:00 CEST 2006
Le samedi 07 octobre 2006 à 21:07 +0100, Martyn Russell a écrit :
> Richard Hult wrote:
> > Xavier Claessens skrev:
> >> Le vendredi 06 octobre 2006 à 23:18 +0200, Xavier Claessens a écrit :
> >>> Ok I can rename to gossip-account-generic. It's not a window nor a
> >>> dialog, it's a widget... So what about gossip-account-jabber-widget ?
> >>>
> >>
> >> Hum, I was renaming the module but I realised:
> >>
> >> GossipAccountGenericWidget
> >> GossipAccountJabberWidget
> >>
> >> Those are too long names. I first named it GossipAccountSettingsJabber
> >> and Martyn agreed it was too long. What do you suggest ? Just replace
> >> "unknown" by "generic" ?
> >
> > I would call them
> >
> > GossipAccountWidgetJabber and GossipAccountWidgetGeneric
> >
> > That is not longer than other stuff we already have.
>
> Yea :)
>
> This is all committed now, thanks Richard for reviewing, I have tested
> the original patch and added some usability to it, including:
>
> - Show a message dialog when registration succeeds not just when it fails.
> - Make the registration success/failure message dialog asynchronous
> (i.e. don't call gtk_dialog_run()).
> - Set the focus to the name of the account when adding a new one.
> - Make the blurb on the settings side of the window slightly more user
> friendly when no account is selected/available.
> - Try to reduce the horrible resizing when no account is selected.
> - Don't allow the 'Register' button to be available when no account is
> selected.
> - Don't show the account name/proxy/auto-connect settings when no
> account is selected.
>
> All-in-all a superb patch, thanks Xavier.
Great !
Don't you have forgotten to "cvs add" new files ?
Xavier.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Ceci est une partie de message
=?ISO-8859-1?Q?num=E9riquement?= =?ISO-8859-1?Q?_sign=E9e?=
Url : http://lists.imendio.com/pipermail/gossip-dev/attachments/20061008/bea73f6b/attachment.pgp
More information about the Gossip-dev
mailing list