Review Request 128038: [libtaskmanager] Stop highlighted window effect in group item

David Edmundson david at davidedmundson.co.uk
Wed Dec 21 00:58:15 UTC 2016


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




applets/taskmanager/package/contents/ui/GroupDialog.qml (line 37)
<https://git.reviewboard.kde.org/r/128038/#comment67968>

    note we've moved to phabricator for review requests, which is why this got ignored.
    Though this was still unexcusable on our side for not commenting in 6 months.
    
    Why are you multiplying anything by devicePixelRatio in this ? 
    The logic is saying "don't be more than a 3rd of the screen width". 
    
    Also, to match the comment in the TODO this should be Screen.desktopAvailableWidth


- David Edmundson


On Dec. 16, 2016, 4:38 a.m., Anthony Fieroni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128038/
> -----------------------------------------------------------
> 
> (Updated Dec. 16, 2016, 4:38 a.m.)
> 
> 
> Review request for Plasma, Kai Uwe Broulik, David Edmundson, and Eike Hein.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> 1. Enable highlighted window, tooltips and grouping
> 2. Open group item tasl by clicking left mouse button
> 3. Move mouse cursor over item in group
> 
> Before:
> 1. Items are samll and tough readable
> 2. Items aren't visible cause a highligthWindow effect
> 3. Clicking on tooltip close button case highlightWindow effect to not stop
> 
> After:
> 1. Items are proof readable
> 2. Items are visible
> 3. HighlightWdindow effect is stopped
> 
> 
> Diffs
> -----
> 
>   applets/taskmanager/package/contents/ui/GroupDialog.qml 829fcf0 
>   applets/taskmanager/package/contents/ui/Task.qml 0a59d53 
>   applets/taskmanager/package/contents/ui/ToolTipWindowMouseArea.qml e3551df 
>   applets/taskmanager/package/contents/ui/main.qml e53b194 
>   applets/taskmanager/plugin/backend.h a3788b0 
>   applets/taskmanager/plugin/backend.cpp 4ef5b88 
> 
> Diff: https://git.reviewboard.kde.org/r/128038/diff/
> 
> 
> Testing
> -------
> 
> Screenshot with expected behavior by clicking left button on group item.
> 
> 
> File Attachments
> ----------------
> 
> Vertical panel
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/12/03/35dd438c-fae2-4daa-ba30-09f70ea3b920__Screenshot_20161203_210914.png
> Horizontal panel
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/12/03/fe7c9cab-bb2a-4d17-85de-1934c81a33dc__Screenshot_20161203_211120.png
> 
> 
> Thanks,
> 
> Anthony Fieroni
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161221/ca987ba6/attachment-0001.html>


More information about the Plasma-devel mailing list