[Bug 244799] adding a new messenger address is a bit confusing

[Jonathan Marten]

https://bugs.kde.org/show_bug.cgi?id=244799


Jonathan Marten <jjm at keelhaul.me.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED




--- Comment #1 from Jonathan Marten <jjm keelhaul me uk>  2012-01-04 11:27:45 ---
Git commit 3c31df07e12ef288e4b5f7484a2ccf52701da533 by Jonathan Marten.
Committed on 04/01/2012 at 12:20.
Pushed by marten into branch 'master'.

Improve appearance and operation of the IM address dialogue.

Instead of editing in place with an item edit delegate, use a
separate add/edit dialogue for the protocol and address.  The
item delegate is only used for showing the standard address
in bold.

Open the item editor dialogue on a double click, so those users
who are accustomed to the double click to change will not be
surprised.

Eliminates the confusion when a user clicks on "Add" and then
thinks "but I didn't ask for AIM, how do I change it?".
REVIEW:103620
GUI:

M  +1    -0    akonadi/contact/CMakeLists.txt
M  +4    -52   akonadi/contact/editor/im/imdelegate.cpp
M  +0    -2    akonadi/contact/editor/im/imdelegate.h
M  +53   -17   akonadi/contact/editor/im/imeditordialog.cpp
M  +2    -0    akonadi/contact/editor/im/imeditordialog.h
A  +84   -0    akonadi/contact/editor/im/imitemdialog.cpp     [License: BSD]
C  +13   -20   akonadi/contact/editor/im/imitemdialog.h [from:
akonadi/contact/editor/im/imeditordialog.h - 062% similarity]     [License:
UNKNOWN]  *

The files marked with a * at the end have a non valid license. Please read:
http://techbase.kde.org/Policies/Licensing_Policy and use the headers which are
listed at that page.


http://commits.kde.org/kdepimlibs/3c31df07e12ef288e4b5f7484a2ccf52701da533

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.