Bug 388865 - "weather-fog" should be day/night neutral
Summary: "weather-fog" should be day/night neutral
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: 2018-01-12 13:58 UTC by Friedrich W. H. Kossebau
Modified: 2018-01-23 14:07 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
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 2018-01-12 13:58:55 UTC
The "wather-fog" icon has no night counter-part by tradition. Both in the minimum icons as noted in the xdg icon spec (https://standards.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html) as well as in the Oxygen icon set.

The Breeze icon set follows here, but has the sun in the symbol. Which makes the icon day-specific though and makes e.g. the weather applet look strange when showing the fog icon for current weather state at night.

So ideally the sun symbol could be dropped from the icon also in the Breeze icon set.
Comment 1 andreas 2018-01-12 14:02:36 UTC
weather-fog and weather-mist are the same so should I remove the sun also in weather-mist?
Comment 2 Friedrich W. H. Kossebau 2018-01-12 14:12:13 UTC
"weather-mist" (deprecated older oxygen name, might be still in use by some apps) should be a sym-link to "weather-fog" (xdg spec defined).

So yes, but it should happen automatically, no? Thanks for looking at it :)
Comment 3 andreas 2018-01-23 14:07:46 UTC
Git commit 3689fbe19c6625f9da4f75e8699168d66b9c74f1 by andreas kainz.
Committed on 23/01/2018 at 14:06.
Pushed by andreask into branch 'master'.

make weather-fog day/night neutral

M  +55   -29   icons-dark/applets/48/weather-mist.svg
M  +55   -29   icons/applets/48/weather-mist.svg

https://commits.kde.org/breeze-icons/3689fbe19c6625f9da4f75e8699168d66b9c74f1