Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use missile icon on radar instead of arrow icon #2068

Closed

Conversation

parracodea
Copy link
Contributor

Unscanned ships and missiles both appear as arrow icons, which can be difficult to differentiate them at a quick glance. Missiles also have large-sized variants, which get closer to the same size as an unscanned ship.

Before

feature_radar_missile_icon

After

feature_radar_missile_icon_new

@oznogon
Copy link
Contributor

oznogon commented Apr 8, 2024

I like the idea of differentiating missile icons from ship icons, but I don't really like the specific icon proposed, especially at small sizes (being two separate shapes makes it "darker"/"noisy"/harder to spot) or for HVLIs (implies a missile with thrust and/or guidance, but isn't).

@nigelkennington
Copy link

Here is a suggested alternative png for the "missile blip".
missile

@nigelkennington
Copy link

nigelkennington commented Apr 8, 2024

Also, because I quite like the idea, here's a bonus one for hvli/unguided blips:

unguided

@parracodea
Copy link
Contributor Author

Closing this due to the icon having poor readability (especially the small missile), and difficulty determining the heading, among other reasons.

@parracodea parracodea closed this Apr 10, 2024
@nigelkennington
Copy link

I understand this be too late as these are closed now, but as a final suggestion: would it be possible to include this but by default have the "missile.png" be a renamed copy of the existing "arrow.png" so that if someone does want to replace them they can do it as part of a mod but those who prefer the traditional blip and arrow will be unaffected?

@parracodea
Copy link
Contributor Author

parracodea commented Apr 11, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants