From kde-panel-devel Wed Sep 30 01:53:08 2015 From: "Takahiro Hashimoto" Date: Wed, 30 Sep 2015 01:53:08 +0000 To: kde-panel-devel Subject: Re: Review Request 125402: [kickerdash] Show ToolTip when mouse hovered on app item Message-Id: <20150930015308.16344.95229 () mimi ! kde ! org> X-MARC-Message: https://marc.info/?l=kde-panel-devel&m=144357800601552 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--===============9034112133942019075==" --===============9034112133942019075== Content-Type: multipart/alternative; boundary="===============1674592417965429308==" --===============1674592417965429308== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125402/ ----------------------------------------------------------- (Updated 9月 30, 2015, 10:53 a.m.) Review request for Plasma and Eike Hein. Changes ------- Add dependency https://git.reviewboard.kde.org/r/125443/ for good UX. I'd like to commit/push this patch after it is resolved. Thanks you all again for great help:) Repository: plasma-desktop Description ------- Now the label of app is elided on Kicker dashboard. It's good, of course. We sometimes want to see full text of app name, for example when the same icon is used for some apps or a app which we see first on the dashboard. PlasmaCore.ToolTipArea is good solution, I think. Diffs ----- applets/kicker/package/contents/ui/ItemGridDelegate.qml 3de7aeb Diff: https://git.reviewboard.kde.org/r/125402/diff/ Testing ------- Build and Test on latest. 1. Open dashboard 2. make mouse hover on app cell. ToolTip is shown and hide on click/press enter on app cell and closing dashboard. File Attachments ---------------- dash_tooltip screenshot https://git.reviewboard.kde.org/media/uploaded/files/2015/09/26/f02936e2-a438-4ec4-9189-10e59fd84965__kickerdash_tooltip.png Tooltip design with tooltipDelegate https://git.reviewboard.kde.org/media/uploaded/files/2015/09/30/10119564-04e1-4a00-a34d-fc3ea8227761__kickerdash_tooltip2.png Thanks, Takahiro Hashimoto --===============1674592417965429308== MIME-Version: 1.0 Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: 8bit

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

Review request for Plasma and Eike Hein.

By Takahiro Hashimoto.

Updated 9月 30, 2015, 10:53 a.m.

Changes

Add dependency https://git.reviewboard.kde.org/r/125443/ for good UX. I'd like to commit/push this patch after it is resolved. Thanks you all again for great help:)

Repository: plasma-desktop

Description

Now the label of app is elided on Kicker dashboard. It's good, of course.

We sometimes want to see full text of app name, for example when the same icon is used for some apps or a app which we see first on the dashboard.

PlasmaCore.ToolTipArea is good solution, I think.

Testing

Build and Test on latest.

Open dashboard
make mouse hover on app cell.

ToolTip is shown and hide on click/press enter on app cell and closing dashboard.

Diffs

applets/kicker/package/contents/ui/ItemGridDelegate.qml (3de7aeb)

File Attachments

dash_tooltip screenshot

Tooltip design with tooltipDelegate

--===============1674592417965429308==-- --===============9034112133942019075== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KUGxhc21hLWRl dmVsIG1haWxpbmcgbGlzdApQbGFzbWEtZGV2ZWxAa2RlLm9yZwpodHRwczovL21haWwua2RlLm9y Zy9tYWlsbWFuL2xpc3RpbmZvL3BsYXNtYS1kZXZlbAo= --===============9034112133942019075==--