Bug 403830

Summary:	weather-storm-* icons show sun/moon in front of cloud
Product:	[Plasma] Breeze	Reporter:	Friedrich W. H. Kossebau <kossebau>
Component:	Icons	Assignee:	visual-bugs-null
Status:	RESOLVED FIXED
Severity:	normal	CC:	kainz.a, nate
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://commits.kde.org/breeze-icons/036f849a12ee455a81f2721196ccb9d07b013486	Version Fixed/Implemented In:	5.55
Sentry Crash Report:

Description Friedrich W. H. Kossebau 2019-02-01 14:08:57 UTC

I feel challenged somehow by seeing the sun/moon cover the second cloud in the background. As much as I see the advantage of showing more of the sun/moon to make the shapes easier to recognize, but perhaps the artistic freedom here could be brought more into line with physical realities, without losing too much of the recognizability?

Start "cuttlefish", type "weather-storm-" and see yourself.

Comment 1 Nate Graham 2019-02-01 20:40:29 UTC

Agreed. Will fix.

Comment 2 Nate Graham 2019-02-01 20:52:40 UTC

Here's a patch: https://phabricator.kde.org/D18652

Comment 3 Nate Graham 2019-02-02 03:45:58 UTC

Git commit 036f849a12ee455a81f2721196ccb9d07b013486 by Nate Graham.
Committed on 02/02/2019 at 03:45.
Pushed by ngraham into branch 'master'.

Improve weather-storm-* icons

Summary:
Current icons are a bit too visually busy and cluttered. This patch corrects that by removing the background cloud and slightly resizing and repositioning the sun and moon.
FIXED-IN: 5.55

Test Plan:
Before: {F6581907, size=full}
After: {F6581952, size=full}

Reviewers: #vdg, kossebau, ndavis

Reviewed By: #vdg, ndavis

Subscribers: ndavis, kde-frameworks-devel

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D18652

M  +58   -54   icons-dark/applets/48/weather-storm-day.svg
M  +60   -55   icons-dark/applets/48/weather-storm-night.svg
M  +58   -54   icons/applets/48/weather-storm-day.svg
M  +60   -55   icons/applets/48/weather-storm-night.svg

https://commits.kde.org/breeze-icons/036f849a12ee455a81f2721196ccb9d07b013486