Skip to content

SpriteFramesEditor Fix preview grid in "Select Frames" dialog - #52461

Merged
akien-mga merged 1 commit into
godotengine:masterfrom
kleonc:sprite-frames-dialog-fix-drawing
Sep 17, 2021
Merged

akien-mga merged 1 commit into
godotengine:masterfrom
kleonc:sprite-frames-dialog-fix-drawing

Conversation

@kleonc

@kleonc kleonc commented Sep 7, 2021

Copy link
Copy Markdown
Member

Fixes #52099 (I'll probably enhance this dialog in another PR, this one is bugfix only).
Fixes #52745.
Cherry-pickable 3.x, 3.3.

Makes grid being drawn according to the frames this dialog outputs (it allows integer-only frame sizes), scaling/zooming wasn't properly taken into account. Input handling is changed accordingly too. Output frames are unchanged, the preview was broken.

Example:

Godot_v3 4-beta4_win64_mGsOKTC3rh
so the frame size is (64, 64) / (13, 11) = (4, 5)

3.4.beta4: this PR (against 3.x):
O93WQ2gDRn QcskcHrE9x
cjo3qlcwIb hvMlLyt1G7
XKQvrIGTt6 jVQhotk0ZF
cVtkUpkptT YuPPnn9f93
pswXOyGA2b WkOCVFARhL
sP3dr1zjyf 0ILJF8EZ9W

@YeldhamDev YeldhamDev added bug topic:editor cherrypick:3.3 cherrypick:3.x Considered for cherry-picking into a future 3.x release labels Sep 7, 2021
@YeldhamDev YeldhamDev added this to the 4.0 milestone Sep 7, 2021
@akien-mga
akien-mga merged commit 87de2e7 into godotengine:master Sep 17, 2021
@akien-mga

Copy link
Copy Markdown
Member

Thanks!

@kleonc
kleonc deleted the sprite-frames-dialog-fix-drawing branch September 18, 2021 07:24
@akien-mga

Copy link
Copy Markdown
Member