Bug 403830 - weather-storm-* icons show sun/moon in front of cloud
Summary: weather-storm-* icons show sun/moon in front of cloud
Status: RESOLVED FIXED
Alias: None
Product: Breeze
Classification: Plasma
Component: Icons (other bugs)
Version First Reported In: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: visual-design
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-02-01 14:08 UTC by Friedrich W. H. Kossebau
Modified: 2019-02-02 03:45 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.55
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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