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
Spritesheet rework #2348
base: develop
Are you sure you want to change the base?
Spritesheet rework #2348
Conversation
7cb859c
to
506a5e2
Compare
static/data/sprites_map.json
Outdated
@@ -0,0 +1,2541 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This shouldn't be in the PR.
.gitignore
Outdated
static/icons/*.png | ||
static/sass/sprite.sass |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should be static/sass/sprite.scss
Missing static/data/sprites_map.json
and static/spritesheet.png
476b13d
to
3439f06
Compare
a790c9c
to
e40d0c6
Compare
Can we use just a "?" on unknown raid boss in same font/size/style as the standard circles/numbers? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using 'rebuild = true' in 'static/js/build_icons.js' overwrite custom icons with default numbers/circles.
Apparently I forgot the 'karp. (Cheers @sirlindqvist 😄 ) |
Yopp, now rebase proper! |
does this handle pokemon forms for spritegen? is it possible to also pull in the sprites for costume from #2461 |
It adds support for Pokemon forms, but it doesn't add support for costumes. That's a possible addition - I'll consider adding it to the PR. But this PR is already quite large (and behind schedule) so that may get delegated to a followup PR. |
Description
This moves all* of the images used by the project into a single spritesheet.
* 'all' doesn't include:
markercluster.js
doesn't support thisMotivation and Context
The current sprite system requires a lot of manual work in order to issue new sprites. This PR aims to make the process for generating sprites much easier.
Additionally, this should make using custom (potentially map-specific) icons much easier.
Shoutout to @sirlindqvist for their help and advice throughout putting this together 🥇 .
How Has This Been Tested?
Small local map.
Screenshots (if appropriate):
Types of changes
Checklist: