Review Request: This patch will check if no accounts will be present then an dialog box will be opened telling no accounts found and giving an option to create on.

[Martin Klapetek]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101362/#review3336
-----------------------------------------------------------



main-widget.cpp
<http://git.reviewboard.kde.org/r/101362/#comment2800>

    You can actually create the whole dialog here, (ie. move the code from top here) and therefore you won't even need the "else {" part. In fact, you won't need even the member variable (the m_noAccountsDialog), you can create it on place.


- Martin


On May 14, 2011, 4:52 p.m., Tarun Mall wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101362/
> -----------------------------------------------------------
> 
> (Updated May 14, 2011, 4:52 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Summary
> -------
> 
> This patch will check if no accounts will be present then an dialog box will be opened telling no accounts found and giving an option to create on.
> 
> 
> Diffs
> -----
> 
>   main-widget.h 7a5e417 
>   main-widget.cpp 20e8003 
> 
> Diff: http://git.reviewboard.kde.org/r/101362/diff
> 
> 
> Testing
> -------
> 
> I tested this patch with
> 1. No accounts
> 2. Creating one account
> 3. Deleting that account and then creating one again.
> 
> 
> Thanks,
> 
> Tarun
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110515/3378eb14/attachment.htm