[Breeze] [Bug 393999] breeze-icons >=5.46.0 fails its scalable test

[Michael Palimaka]

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

Michael Palimaka  changed:

   What|Removed |Added

 CC||kensing...@gentoo.org

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

[Breeze] [Bug 393999] breeze-icons >=5.46.0 fails its scalable test

[David Faure]

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

David Faure  changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/bre
   ||eze-icons/e7330e352bb17e732
   ||0c0b16f9aa32fe70384788e
 Status|UNCONFIRMED |RESOLVED
   Version Fixed In||5.49
 Resolution|--- |FIXED

--- Comment #4 from David Faure  ---
Git commit e7330e352bb17e7320c0b16f9aa32fe70384788e by David Faure, on behalf
of Maximiliano Curia.
Committed on 08/08/2018 at 09:44.
Pushed by dfaure into branch 'master'.

ScalableTest, add "scalable" plasma-browser-integration

With the introduction of the plasma-browser-integration icon (Add
plasma-browser-integration system tray icon, 47b8e71a) the ScalableTest started
failing [1]. The test seems to expect a "larger" icon (greater than 48)
available to consider that there is a scalable icon. I'm not
sure if this test is still useful, as all the icons are now in svg format, and
while they might be designed for a low resolution, they are "scalable".

This patch simply adds symlinks to the 48 and 64 res, in order to make the
test pass.

[1]:
https://build.kde.org/job/Frameworks%20breeze-icons%20kf5-qt5%20SUSEQt5.10/81/
FIXED-IN: 5.49

A  +1-0icons-dark/apps/48/plasma-browser-integration.svg
A  +1-0icons-dark/apps/64/plasma-browser-integration.svg
A  +1-0icons/apps/48/plasma-browser-integration.svg
A  +1-0icons/apps/64/plasma-browser-integration.svg

https://commits.kde.org/breeze-icons/e7330e352bb17e7320c0b16f9aa32fe70384788e

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

[Breeze] [Bug 393999] breeze-icons >=5.46.0 fails its scalable test

[Christoph Feck]

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

--- Comment #3 from Christoph Feck  ---
So if I understand the issue correctly, we would need to make a whitelist for
icons that do not need to be scalable to arbitrary sizes, and exclude them from
the tests that check scalability?

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

[Breeze] [Bug 393999] breeze-icons >=5.46.0 fails its scalable test

[andreas_k]

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

andreas_k  changed:

   What|Removed |Added

 CC||kaiuwebroul...@hotmail.com

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

[Breeze] [Bug 393999] breeze-icons >=5.46.0 fails its scalable test

[andreas_k]

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

andreas_k  changed:

   What|Removed |Added

 CC||andrea...@abwesend.de

--- Comment #2 from andreas_k  ---
kai want's the system tray icon in breeze-icons and it look that there is no
larger app icon needed, so I'm not interested in draw one when it's not needed.

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

[Breeze] [Bug 393999] breeze-icons >=5.46.0 fails its scalable test

[Christoph Feck]

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

--- Comment #1 from Christoph Feck  ---
Andreas, any idea how to resolve this issue?

Additionally, are systray icons part of the icon theme or the workspace theme?

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

[Breeze] [Bug 393999] breeze-icons >=5.46.0 fails its scalable test

[Nate Graham]

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

Nate Graham  changed:

   What|Removed |Added

 CC||n...@kde.org
   Assignee|visual-des...@kde.org   |kain...@gmail.com

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