https://bugs.kde.org/show_bug.cgi?id=525806

Arjen Hiemstra <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REPORTED                    |CONFIRMED
     Ever confirmed|0                           |1

--- Comment #1 from Arjen Hiemstra <[email protected]> ---
The problem with these is that URLs can be *anything*, including remote images.
I wanted to avoid the CSS style specifying remote URLs as that would lead to
potential security issues. However, I've since realised that we should filter
that on the input side as Icon is also used by applications which should be
able to specify remote URLs or anything else.

> The Breeze/Desktop styles rendered those icons at least on  `RoundButton` and 
> `MenuItem`. So perhaps the code from these styles can be reused here.

Kirigami.Icon, which is what renders these things for those styles, does have
code but it's a lot of complex state machinery that I really wanted to avoid
having to copy over. But it might be possible to create something that supports
most of this without the amount of code that's in Kirigami.Icon.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to