← Back to team overview

genesis-devs team mailing list archive

Re: Account management [was: gtk.Builder migration]

 

Hi!

I've had a very busy weekend (two birthdays the Saturday, a
Sunday-morning travel, an Internet connection problem...)  and I
couldn't dedicate to SE all the time I wanted. Luckily I was able to
spend a couple of hours implementing a few things related to Account
Management, which I would like to comment.

About the sync drop-down, you really convinced me with your arguments
about the buttons vs radio thing and I've pushed to my bzr branch
another version with your option. It's still not functional, it's only
a elaborated mockup, but when I have time I'll complete it. But I've
been thinking about it this week, and I think the best we can do with
the drop-down menu is to delete it from the toolbar and keep the plain
'two-way sync' button. If a user want to perform a special sync, the
option is still available in the menubar, under ''Acount > Synchronize
> XXXX". I think this way is simpler than both previous proposals...
what do you think about it?

I've pushed the support for toggling on/off the automatic
synchronization for an account from the 'Account Management' screen,
I've implemented it the way you suggested (saving the account names in
the configuration file). It still needs some love, but it's currently
functional. I've encountered some problems:
- I've added a new column with a toggle button which control the
automatic synchronization status for the account. But I don't know
what name to give it... for example "Synchronize automatically" is too
long... and the checkbox looks weird with such big space. For now I've
leave it without title... Can you think a better column title, or
should we left it blank? Mmm, "auto sync" is weird too :-S
- I've added a new configuration key, called 'autosync_servers', which
store a list of accounts names. The problem is that I've been forced
to violate the encapsulation of GenesisConfig class, calling
'config.write()' directly from ManageAccountsWindow class, because
when adding/removing elements from the list the method __setitem__
from GenesisConfig isn't called.... maybe I've not explained it too
well (my English isn't good enough to explain that properly). What can
we do, should we refactor GenesisConfig? There are some remaining
glitches, but I'll work on them shortly.

For now I do not remember any more troubles, but there was more for
sure. When I recall them, I'll write to the list again.

Cheers!!! :-)



Follow ups

References