Bug 364981 - Some breeze icons for KStars are inappropiate
Summary: Some breeze icons for KStars are inappropiate
Status: RESOLVED FIXED
Alias: None
Product: Breeze
Classification: Plasma
Component: Icons (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: andreas
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-07-01 17:01 UTC by Jasem Mutlaq
Modified: 2016-12-11 23:09 UTC (History)
3 users (show)

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


Attachments
Problematic Breeze icons in KStars (34.72 KB, image/png)
2016-07-01 17:01 UTC, Jasem Mutlaq
Details
icons previously used before for constellation art (1.79 KB, image/png)
2016-07-01 17:05 UTC, Jasem Mutlaq
Details
kstars screenshot (135.59 KB, image/png)
2016-09-30 07:27 UTC, andreas
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jasem Mutlaq 2016-07-01 17:01:09 UTC
First of all, we greatly appreciate the effort when into making Breeze icons for KStars. It's beautiful!

However, some of the icons are inappropriate. I'm attaching a screenshot with arrows at the icons we'd like to change:

1. Star: While the icon shows a single star, it would be more intuitive to check if you can multiple stars icons somehow.
2. Deep Sky Object: A circle is not representative alone. Maybe something with the shape of a galaxy (but should be different from FITSViewer icon), or nebula.. Something that clearly signifies a deep sky object.
3. Solar System: Current icon shows earth. While this is somewhat acceptable, it would be better if you can swap it with an image of a prominent recognizable solar system body like Saturn
4. Supernova: This one does not make sense at all. Supernova is a stellar explosion, no two buddies on chat. So please try to come up with something that represents this.

Again all your efforts are greatly appreciated!

Reproducible: Always
Comment 1 Jasem Mutlaq 2016-07-01 17:01:51 UTC
Created attachment 99789 [details]
Problematic Breeze icons in KStars
Comment 2 Jasem Mutlaq 2016-07-01 17:04:14 UTC
Please add to the list:

5. Constellar Art: The icon is completely inappropriate. Please take a look at the one used by KStars.
Comment 3 Jasem Mutlaq 2016-07-01 17:05:37 UTC
Created attachment 99790 [details]
icons previously used before for constellation art
Comment 4 andreas 2016-07-02 06:50:23 UTC
Thanks for the feedback will update them
Comment 5 Jasem Mutlaq 2016-08-07 13:13:08 UTC
Hello. Any update on this?
Comment 6 andreas 2016-08-07 20:18:19 UTC
thought I already updated them. sorry will have a look the next days
Comment 7 Jasem Mutlaq 2016-08-25 08:48:02 UTC
Sorry to bring this up again but I didn't see any updates, is this on git master or somewhere else?
Comment 8 Jasem Mutlaq 2016-09-21 18:57:50 UTC
Hello, any update on this? I realize you guys are busy, but if there this cannot be done, then we will include our own icons in the resource file and discard breeze icons.
Comment 9 Alex L 2016-09-23 08:51:23 UTC
Hi Jasem, thanks for reporting this.
I have zero knowledge about Kstar, could you please help describing the 5 actions with the wrong icons? Like "Star do...".
Also, maybe you have a screenshot with the previous icons?
Comment 10 Jasem Mutlaq 2016-09-23 09:30:56 UTC
Hi Alex, here is a previous screenshot: https://edu.kde.org/kstars/screens5.x/kstars_billionsandbillions.png
Comment 11 Jasem Mutlaq 2016-09-23 09:38:04 UTC
I already described what's wrong with the Breeze icons in my initial post. The icons do not represent the objects they are supposed to represent. The problematic icons in Breeze are (names):

1. kstars_stars
2. kstars_deepsky
3. kstars_planets
4. kstars_constellationart
5. kstars_supernovae
Comment 12 andreas 2016-09-30 07:14:11 UTC
Git commit 58913d5b95eb463b7dc168c4ec6e7927410de3f4 by andreas kainz.
Committed on 30/09/2016 at 07:13.
Pushed by andreask into branch 'master'.

update some kstars icons

T  +19   -1    icons/actions/22/kstars_constellationart.svg
M  +2    -2    icons/actions/22/kstars_deepsky.svg
T  +22   -1    icons/actions/22/kstars_planets.svg
T  +14   -1    icons/actions/22/kstars_stars.svg
T  +22   -1    icons/actions/22/kstars_supernovae.svg

http://commits.kde.org/breeze-icons/58913d5b95eb463b7dc168c4ec6e7927410de3f4
Comment 13 andreas 2016-09-30 07:27:47 UTC
Created attachment 101356 [details]
kstars screenshot

this bug isn't fixed cause I need some input for the constellation icons

- kstars_clines
show the constellation lines of Big Dipper

- kstars_cnames
say Ori is the big dipper part of orion? if no, I'd like to add the acronym for big dipper (-> how is it called? I can write 3 letters)

- kstars_constellationart
I know wrong icon. I'd like to draw the big dipper (need some time)

the final result should be to see first the lines than the name and than the artwork of the same element (big dipper). is big dipper also in the artwork view of kstars? if yes, please send a screenshot.
Comment 14 Jasem Mutlaq 2016-09-30 08:43:26 UTC
Thanks for the great work!

- kstars_clines
- It is fine

- kstars_cnames
- Well, it's Ursa Major. Maybe it can be kept as is, Orion is a completely different constellation, but the idea is to convey "click here for text of constellations"

- kstars_constellationart
- Well, it doesn't have to be big dipper. It can be any shape that convey "image". For example, a bow and arrow is sufficient (e.g. http://www.clipartbest.com/cliparts/xig/Kea/xigKea64T.png)

Only thing remaining is supernova, the icon doesn't convey that at all, a super nova is a stellar explosion, so a star exploding into oblivion.
Comment 15 andreas 2016-10-14 07:21:08 UTC
Git commit 95d0046dc25068ee965e50dfb6830f3413e67345 by andreas kainz.
Committed on 14/10/2016 at 07:19.
Pushed by andreask into branch 'master'.

update kstars action icons

T  +14   -1    icons-dark/actions/22/kstars_constellationart.svg
M  +2    -7    icons/actions/22/kstars_constellationart.svg
T  +1    -22   icons/actions/22/kstars_planets.svg
M  +69   -13   icons/actions/22/kstars_supernovae.svg
A  +149  -0    icons/mimetypes/64/text-x-sass.svg
A  +1    -0    icons/mimetypes/64/text-x-scss.svg
A  +149  -0    icons/mimetypes/text-x-sass.svg

http://commits.kde.org/breeze-icons/95d0046dc25068ee965e50dfb6830f3413e67345
Comment 16 andreas 2016-10-14 07:22:02 UTC
should work now isn't it?
Comment 17 Jasem Mutlaq 2016-10-14 09:07:22 UTC
Thanks a lot for your work, they are beautiful!
Comment 18 Jasem Mutlaq 2016-12-03 15:00:22 UTC
I just switched to Breeze dark theme and the old icons are still there. Can you please also update the dark theme icons as well?
Comment 19 andreas 2016-12-11 23:09:13 UTC
sorry I was lazy now breeze and breeze-dark are synced