Skip to content

Fix scaling bug on FlxView3D on High DPI displays - #665

Open
kitzsh wants to merge 2 commits into
CodenameCrew:mainfrom
StrawberrySage:away3d-highdpi-fix
Open

Fix scaling bug on FlxView3D on High DPI displays#665
kitzsh wants to merge 2 commits into
CodenameCrew:mainfrom
StrawberrySage:away3d-highdpi-fix

Conversation

@kitzsh

@kitzsh kitzsh commented Jun 30, 2025

Copy link
Copy Markdown
Contributor

Fixes a bug in FlxView3D where the underlying View3D is larger than the BitmapData it renders to when "allow-high-dpi" is enabled, only showing the top left-hand corner of the view.

@kitzsh
kitzsh marked this pull request as draft June 30, 2025 03:22
@kitzsh

kitzsh commented Jun 30, 2025

Copy link
Copy Markdown
Contributor Author

Needs more testing actually

@kitzsh

kitzsh commented Jun 30, 2025

Copy link
Copy Markdown
Contributor Author

Tested it with a few different resolutions, should be all clear

@kitzsh
kitzsh marked this pull request as ready for review June 30, 2025 03:33
@crowplexus crowplexus added the type:bug Something isn't working label Jun 30, 2025
@crowplexus

Copy link
Copy Markdown
Member

OH SO THAT'S WHY????

@kitzsh

kitzsh commented Jun 30, 2025

Copy link
Copy Markdown
Contributor Author

yup

@TechnikTil TechnikTil left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ohhhh
so thats why

@NexIsDumb

Copy link
Copy Markdown
Member

was this actually tested enough?

@Raltyro

Raltyro commented Aug 9, 2025

Copy link
Copy Markdown
Member

There has to be another issue relating to dpi, because texts just gets messed up if the current game resolution isnt in default

@Raltyro Raltyro added the status:invalid This doesn't seem right label Aug 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:invalid This doesn't seem right type:bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants