> On Jan. 18, 2014, 1:20 a.m., Martin Klapetek wrote:
> > File Attachment: kcm.png - kcm.png
> > <https://git.reviewboard.kde.org/r/115076/#fcomment130>
> >
> >     I'm not sure if duplicating this is a good idea...the rest yes, this 
> > button, not so sure.

Genereally I don't think duplicating things is a problem. What might be a 
problem here is that users might have difficulties finding the regular Add 
Account button afterwards because they keep looking for it somewhere in the 
accounts list. With QML, we could just put a big red arrow pointing to the 
button in there instead of the duplicate button, but it might be difficult with 
widgets...


- Thomas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115076/#review47622
-----------------------------------------------------------


On Jan. 17, 2014, 2:56 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115076/
> -----------------------------------------------------------
> 
> (Updated Jan. 17, 2014, 2:56 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Repository: ktp-accounts-kcm
> 
> 
> Description
> -------
> 
> Display nice UI when no accounts are present, see screenshot
> 
> 
> Diffs
> -----
> 
>   src/CMakeLists.txt cf7998e 
>   src/accounts-list-view.h PRE-CREATION 
>   src/accounts-list-view.cpp PRE-CREATION 
>   src/kcm-telepathy-accounts.cpp c8b837b 
>   src/main-widget.ui 4983c71 
> 
> Diff: https://git.reviewboard.kde.org/r/115076/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> kcm.png
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2014/01/17/2880d55c-2c75-4706-9694-ef25fdd92f47__kcm.png
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

_______________________________________________
KDE-Telepathy mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-telepathy

Reply via email to