On Thursday 04 October 2007 19:02:15 Dennis Nienhüser wrote: > Matt Rogers schrieb: > > On Thursday 04 October 2007 06:11:14 Dennis Nienhüser wrote: > >> Hi, > >> > >> when I used the account manager the first time it confused me quite a > >> bit. I think we could improve the first time experience with some simple > >> usability tweaks. I made a first mockup showing things that bother me > >> and how they could be improved, see > >> http://dev.gentoo.org/~earthwings/misc/am.png > >> Please leave some comments. > >> > >> Dennis > > > > Looks great. I say go ahead and implement this and send in the patch and > > we'll get it committed. :) > > Sounds good, patch is attached. It's a bit longer this time because I > also added the context menu, which required some refactoring prior to > this to have KActions do the job for less duplication. > > I changed the following behavior: > - You can only switch identities if you have more than one > - You cannot delete the default identity anymore. This is especially > useful to avoid removing the last identity which contains accounts, > otherwise a non-completable dialog pops up. > > Please test whether I broke something when trying to fix these things: > - When removing an identity containing accounts, the TreeWidget lost > them. I added another load() call in the end of the function to avoid that > - Account status changes were not displayed as the tree widget was > searched flat (I guess it was a list before) > > It's still possible to drag 'n drop an account to the virtual tree root. > I think it's ok for now though (identity is kept, it's only visually > annoying). In the long run, a switch to MVC would be best and fix this > as a side-effect. > > Screenshot is at http://dev.gentoo.org/~earthwings/misc/am1.png > > Regards, > Dennis
Just out of curiousity, if we have the buttons on the side, why do we need a context menu? -- Matt _______________________________________________ kopete-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kopete-devel
