Bug 296655 - JJ: Add tooltip (patch attached)
Summary: JJ: Add tooltip (patch attached)
Status: RESOLVED FIXED
Alias: None
Product: telepathy
Classification: Unmaintained
Component: chat-plasmoid (show other bugs)
Version: git-latest
Platform: Gentoo Packages Linux
: NOR wishlist
Target Milestone: 0.5-next
Assignee: Telepathy Bugs
URL:
Keywords: junior-jobs
Depends on:
Blocks:
 
Reported: 2012-03-24 01:51 UTC by Elias Probst
Modified: 2012-08-26 18:03 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Adds a Tooltipp with name and presence icon. (3.50 KB, patch)
2012-05-13 22:31 UTC, Michael Bohlender
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Elias Probst 2012-03-24 01:51:15 UTC
When hovering a contact in the chat-plasmoid, a tooltip should be shown showing the following information:

- User name
- Status (Online, Offline, Away, …) as icon
- Presence message
Comment 1 Michael Bohlender 2012-05-13 14:53:40 UTC
User name is done, i ll try to add Status and Presence message.
Comment 2 Michael Bohlender 2012-05-13 20:58:38 UTC
 Tooltip with status icon and user name: 

http://paste.kde.org/477380/
http://paste.kde.org/477392/
http://paste.kde.org/477386/

PlasmaCore.ToolTip would only take an icon location and not an icon so i had to change the model.
I fixed the icon size at 44x44.
I dont know if that were good decisions. Its the first time i try to do something usefull with cpp/qt/qml.
Critique is wellcome :)
Comment 3 Michael Bohlender 2012-05-13 22:12:56 UTC
Improved version:
http://paste.kde.org/477470/
http://paste.kde.org/477476/
http://paste.kde.org/477482/
Comment 4 Martin Klapetek 2012-05-13 22:15:10 UTC
If you did not do any commits, you can do simply "git diff", that will get you a complete diff in one piece. Also please use the attachments in here as pastes will expire while the attachments will stay here.
Comment 5 Michael Bohlender 2012-05-13 22:31:22 UTC
Created attachment 71076 [details]
Adds a Tooltipp with name and presence icon.
Comment 6 David Edmundson 2012-06-26 18:19:43 UTC
Git commit 77704d02ed38724db3f71e2987a029c77a6358ee by David Edmundson, on behalf of Michael Bohlender.
Committed on 26/06/2012 at 20:17.
Pushed by davidedmundson into branch 'master'.

Add tooltip to quick chat plasmoid

Reviewed-by: David Edmundson

M  +10   -0    plasmoid/declarative-plugin/conversation-target.cpp
M  +3    -0    plasmoid/declarative-plugin/conversation-target.h
M  +7    -0    plasmoid/org.kde.ktp-chatplasmoid/contents/ui/ConversationDelegate.qml

http://commits.kde.org/telepathy-text-ui/77704d02ed38724db3f71e2987a029c77a6358ee
Comment 7 David Edmundson 2012-06-26 18:20:37 UTC
Sorry about the delay! 
There was the freeze for 0.4, and then I forgot.
Applied, hope to see more patches from you in the future.
Comment 8 Adrián Chaves (Gallaecio) 2012-08-26 18:03:20 UTC
I guess this can be closed.