Review Request 128573: Add overlay support to PlasmaCore::IconItem

David Edmundson david at davidedmundson.co.uk
Tue Aug 2 10:08:00 UTC 2016


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




src/declarativeimports/core/iconitem.cpp (line 222)
<https://git.reviewboard.kde.org/r/128573/#comment66015>

    you're going to need some to signal that the icon needs to be redrawn
    
    otherwise if you set the overlay after the initial component is loaded it won't work.
    
    add "schedulePixmapUpdate()" somewhere in this method



src/declarativeimports/core/iconitem.cpp (line 472)
<https://git.reviewboard.kde.org/r/128573/#comment66016>

    why loop if we're going to use m_overlay every time?


- David Edmundson


On Aug. 2, 2016, 8:24 a.m., Chinmoy Ranjan Pradhan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128573/
> -----------------------------------------------------------
> 
> (Updated Aug. 2, 2016, 8:24 a.m.)
> 
> 
> Review request for Plasma and Eike Hein.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> As suggested in https://git.reviewboard.kde.org/r/128552/ (by Eike), this patch adds overlay support to IconItem.
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/core/iconitem.h 6a1d67c 
>   src/declarativeimports/core/iconitem.cpp fae0330 
> 
> Diff: https://git.reviewboard.kde.org/r/128573/diff/
> 
> 
> Testing
> -------
> 
> build
> 
> 
> Thanks,
> 
> Chinmoy Ranjan Pradhan
> 
>

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


More information about the Plasma-devel mailing list