Skip to content
New issue

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

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

Already on GitHub? # to your account

Convert images to SVG #35

Closed

Conversation

microlith57
Copy link
Contributor

@microlith57 microlith57 commented Oct 3, 2019

Closes #34 by converting all images to SVG using line tracing.

Progress:

  • Convert images, change code references to be .svg
  • Make theme affect SVG colours on a per-<img> basis (so that non-changing images can be used in parallel)
  • Figure out ASAR archives?

Note: This PR also adds some documentation in the action code to make it more accessible to new developers :)

This process used Inkscape's bitmap tracing, roughly halving total image file size :)

Signed-off-by: microlith57 <microlith57@gmail.com>
@microlith57
Copy link
Contributor Author

It might also be good to move all the assets into one location so they can be used by both desktop and web.

- SVGs change colour with theme changes (`f_high`)
- Fixed issue with `learn` and `help` SVGs having weird outline when 
themed

Signed-off-by: microlith57 <microlith57@gmail.com>
@microlith57 microlith57 marked this pull request as ready for review October 4, 2019 04:09
@microlith57
Copy link
Contributor Author

@neauoire Added theming :) Just need to test bundle compatibility.

@microlith57
Copy link
Contributor Author

microlith57 commented Oct 4, 2019

Oh no; it doesn't work on mac when bundled!

Edit: I can use path.resolve for this.

Hopefully!

Signed-off-by: microlith57 <microlith57@gmail.com>
Signed-off-by: microlith57 <microlith57@gmail.com>
@microlith57
Copy link
Contributor Author

@neauoire Ready for review! :)

@neauoire neauoire closed this Dec 28, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Convert learn images to SVG
2 participants