Skip to content

Always use SVG mipmaps that match or exceed the skins' screen-space size #649

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

Merged
merged 2 commits into from
Aug 4, 2020

Conversation

adroitwhiz
Copy link
Contributor

Resolves

Resolves #533
Resolves #620

Proposed Changes

This PR changes the SVG mipmap logic to always use a mipmap with a greater or equal resolution to that of the sprite as displayed on-screen.

Reason for Changes

This makes vector sprites appear less blurry.

Test Coverage

To be tested manually

@fsih fsih added this to the June 2020 milestone Jun 23, 2020
Copy link
Contributor

@cwillisf cwillisf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM! Thanks for making the math easier to read, too :)

Copy link
Contributor

@fsih fsih left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! LG from both me and Chris if it passes performance testing ok

@fsih
Copy link
Contributor

fsih commented Jul 9, 2020

@rschamp rschamp modified the milestones: June 2020, July 2020 Jul 15, 2020
@fsih fsih merged commit 46274c7 into scratchfoundation:develop Aug 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants