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

Icons test #1394

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Icons test #1394

wants to merge 3 commits into from

Conversation

elgandoz
Copy link
Contributor

@elgandoz elgandoz commented Apr 4, 2024

Test Icon PR - WIP

@elgandoz elgandoz force-pushed the new-icons branch 2 times, most recently from 5109ad2 to 97d62d0 Compare April 4, 2024 16:26
@lordfolken
Copy link
Contributor

@Sundown3867 can you have a look please?

@lordfolken
Copy link
Contributor

We really need an icon guideline in the development manual, can one of you write this?

@elgandoz elgandoz changed the title New icons Icons test Apr 4, 2024
@elgandoz
Copy link
Contributor Author

elgandoz commented Apr 4, 2024

Hey @lordfolken, I have the intention to start a GitHub "discussion" about this, and most definitely also include @Sundown3867 and @MikeBravo15 for collaboration, since they already did a great job.
I'm currently editing on a Figma file, outputting "pure" SVG (no Inkscape meta) with simple, optimised vector paths.
I'm trying to address/ understand the sizing issue (testing on Android & Linux) and I'm also trying to improve readability and aesthetics (of the waypoints , for now).
I'm just testing for now, I will start a Discussion soon with all the details.

@lordfolken
Copy link
Contributor

I'm currently editing on a Figma file, outputting "pure" SVG (no Inkscape meta) with simple, optimised vector paths.

+1
Very good I like the no metadata approach. I think @Sundown3867 already did quite a bit of cleanup there.

@elgandoz elgandoz force-pushed the new-icons branch 3 times, most recently from 935c541 to 9b61ce8 Compare April 9, 2024 15:58
@lordfolken
Copy link
Contributor

You marked this as a draft. Are you going to go on? I'd hate to see the effort wasted...

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.

None yet

2 participants