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

Select frames window shows misaligned grid at certain zoom levels #52745

Closed
berarma opened this issue Sep 16, 2021 · 3 comments · Fixed by #52461
Closed

Select frames window shows misaligned grid at certain zoom levels #52745

berarma opened this issue Sep 16, 2021 · 3 comments · Fixed by #52461

Comments

@berarma
Copy link
Contributor

berarma commented Sep 16, 2021

Godot version

v3.4.beta4.official.6a058cbf3

System information

Debian GNU/Linux 11

Issue description

When using the zoom controls in the "Select frames" window (AnimatedSprite node), the grid gets misaligned with the sprites at some zoom levels. It's really easy to see with wide sprite sheets because the error adds up for every sprite in the horizontal direction. At zoom level 1 it looks aligned, and I think that for every integer scale value too, but other values misalign the grid and the sprites.

Steps to reproduce

Take a wide sprite sheet (I've used one with 20 sprites, 16 bytes wide each) so the grid positioning error is more evident. Set the horizontal and vertical counts correctly for the sprite sheet and zoom in. You'll see the grid misalining towards the right side of the sprite sheet.

Minimal reproduction project

No response

@kleonc
Copy link
Member

kleonc commented Sep 16, 2021

I've fixed it lately, check out #52461 (it's not merged yet though).

@berarma
Copy link
Contributor Author

berarma commented Sep 16, 2021

I've fixed it lately, check out #52461 (it's not merged yet though).

Thanks! I couldn't find it in the issues list and didn't think about searching in the PR. I'll try it as soon as it's merged and released.

@Calinou
Copy link
Member

Calinou commented Sep 16, 2021

Duplicate of #52099.

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

Successfully merging a pull request may close this issue.

3 participants