Review Request 111088: [logviewer] Show a message when no logs are available for given contact or day

Dan Vrátil dvratil at redhat.com
Tue Jun 18 16:26:59 UTC 2013



> On June 18, 2013, 4:34 p.m., David Edmundson wrote:
> > You can do adiumThemeView->setHTML it is a webview after all.. but I'm happy with the label.
> > 
> > Have you considered a KMessageWidget, do you think that would be better?

With setHTML() I'd lost the "Next/Previous Conversation" links and the native theme background.

I tried KMessageWidget but it looked a bit weird :-)


- Dan


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


On June 18, 2013, 1:49 p.m., Dan Vrátil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111088/
> -----------------------------------------------------------
> 
> (Updated June 18, 2013, 1:49 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Shows a label saying "There are no logs for this day" or "There are no logs for this contact" when user selects a date without logs or starts logviewer with ID of a contact for which we have no logs.
> 
> The QLabel feels a bit hackish, but with Adium I don't see any other way. We can do this properly once QML viewer is in place.
> 
> 
> This addresses bug 316487.
>     http://bugs.kde.org/show_bug.cgi?id=316487
> 
> 
> Diffs
> -----
> 
>   logviewer/entity-view.h d55f194 
>   logviewer/entity-view.cpp 69c408f 
>   logviewer/log-viewer.h 9341064 
>   logviewer/log-viewer.cpp 743cda4 
>   logviewer/message-view.h c9ae322 
>   logviewer/message-view.cpp 96ad846 
> 
> Diff: http://git.reviewboard.kde.org/r/111088/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> 
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/06/18/logviewer.png
> 
> 
> Thanks,
> 
> Dan Vrátil
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130618/7d28ec7d/attachment.html>


More information about the KDE-Telepathy mailing list