Summary: | Wrong icon name: weather-none should be weather-none-available | ||
---|---|---|---|
Product: | [Plasma] Breeze | Reporter: | Friedrich W. H. Kossebau <kossebau> |
Component: | Icons | Assignee: | visual-bugs-null |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | kainz.a |
Priority: | NOR | ||
Version First Reported In: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://commits.kde.org/breeze-icons/87cef4dd7856137e607137241c139f3ee3dbff75 | Version Fixed In: | |
Sentry Crash Report: |
Description
Friedrich W. H. Kossebau
2017-05-15 20:56:12 UTC
Git commit 87cef4dd7856137e607137241c139f3ee3dbff75 by Friedrich W. H. Kossebau. Committed on 05/01/2018 at 21:54. Pushed by kossebau into branch 'master'. Fix icon name "weather-none" -> "weather-none-available" Summary: "weather-none-available" has been the traditional name in Oxygen and is referenced as such in existing KDE and non-KDE code, also used as name in other icon themes (e.g. Numix). One could argue that the icon fallback mechanism should still result in "weather-none" being used with Breeze. Issues with that though are: * is inconsistent naming with other icon themes (incl. Oxygen) * fails with distributions patching the Breeze theme to inherit Oxygen, as then the Oxygen icon will be used. Reviewers: #frameworks, #breeze, andreask Tags: #frameworks Differential Revision: https://phabricator.kde.org/D9626 R +0 -0 icons-dark/applets/48/weather-none-available.svg [from: icons-dark/applets/48/weather-none.svg - 100% similarity] R +0 -0 icons/applets/48/weather-none-available.svg [from: icons/applets/48/weather-none.svg - 100% similarity] https://commits.kde.org/breeze-icons/87cef4dd7856137e607137241c139f3ee3dbff75 |