Review Request: Kickoff-qml fixes
Sebastian Kügler
sebas at kde.org
Wed Oct 24 21:16:34 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107026/#review20823
-----------------------------------------------------------
Ship it!
One issue (inline), otherwise it looks good.
plasma/desktop/applets/kickoff/package/contents/ui/KickoffItem.qml
<http://git.reviewboard.kde.org/r/107026/#comment16439>
Why remove this fade in animation?
- Sebastian Kügler
On Oct. 24, 2012, 6:20 p.m., David Edmundson wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107026/
> -----------------------------------------------------------
>
> (Updated Oct. 24, 2012, 6:20 p.m.)
>
>
> Review request for Plasma.
>
>
> Description
> -------
>
> If an application has no generic name return the application name as the title.
>
> In this case we return nothing as the subtitle. If the subtitle is the same as the new title, hide the subtitle.
> This fixes a bug in which "Find Files" would show with a subtitle but no title as well as "Utilities" having a pointless subtitle
> --
> Use standard desktop scrolling
> --
> Always show application subtitle.
>
> This makes it the same as previous kickoff. This subtitle is quite important when you have two web browsers (for example)
> --
> Increase padding on LHS between icon and text
>
>
> Diffs
> -----
>
> plasma/desktop/applets/kickoff/core/applicationmodel.cpp fa5ceea58c4e7e27df1a65d5278510b2d0225ffa
> plasma/desktop/applets/kickoff/package/contents/ui/ApplicationsView.qml 4488f2eaea0bc7447318ca341c5b27d6a4eee700
> plasma/desktop/applets/kickoff/package/contents/ui/BaseView.qml 2b1e598cc72a84639ecd467ed558276e972a3510
> plasma/desktop/applets/kickoff/package/contents/ui/KickoffItem.qml 9d468b2e596410eae18aba357bd6307b70758e33
>
> Diff: http://git.reviewboard.kde.org/r/107026/diff/
>
>
> Testing
> -------
>
>
> Screenshots
> -----------
>
> After
> http://git.reviewboard.kde.org/r/107026/s/799/
>
>
> Thanks,
>
> David Edmundson
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20121024/dbd99ee5/attachment.html>
More information about the Plasma-devel
mailing list