On Oct 20, 2007, at 1:48 AM, Will Stephenson wrote: > Hi > > Some usability changes around identities, and related minor libkopete > refactoring: > > * change identityId -> id for clarity > * use immutable id as internal handle > * introduce user editable Label field for use in UI - these 2 > changes allow > the user more control and insulate the user from the implementation > * use pointer based default identity > > * replace dialog for selecting what the new identity will be based > on with two > buttons in the main accountconfig widget - Add Identity and Copy > Identity > (KMail will make this change too) > * adapt dbus interface to use ids, provide mapping method for id- > >label > > There is no config upgrade script for the previous impl so expect your > existing identities to be without labels on first run! > > Will
First impressions: 1. We need a kconf_update script. Leaving users' configuration high and dry is not acceptable anymore. There has been a release with this change in it (even if it is a beta) and we should be moving the user's configuration over. Not moving config is not an option anymore, IMHO. 2. There are no screenshots for the changes. So, I suggest one of two things. Screenshots :) or a set of instructions that describe how to get to the changes. Sure, some of these might be obvious, but we should know what we're looking for since as part of the review process, we should be reviewing the changes to the user interface as well. Could you provide either screenshots or directions on how to see the changed functionality? I have no checkout built with which to test ATM, so I'll test and get back to you. -- Matt _______________________________________________ kopete-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kopete-devel
