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

horizontal game canvas rescale regression #1444

Closed
DreadKnight opened this issue Aug 27, 2018 · 4 comments
Closed

horizontal game canvas rescale regression #1444

DreadKnight opened this issue Aug 27, 2018 · 4 comments
Labels
coding This issue requires some programming easy Good place to start if you're a new developer priority This should get fixed as soon as possible! visuals Various things that easily catch the eye

Comments

@DreadKnight
Copy link
Member

The development version doesn't properly maintain the aspect ratio of the game canvas when making the browser window smaller horizontally, while the stable version works fine. Yet another regression.

screenshot from 2018-08-27 10-15-32
dev: https://ancientbeast-beta.herokuapp.com

screenshot from 2018-08-27 10-15-47

stable: https://play.ancientbeast.com

@DreadKnight DreadKnight added coding This issue requires some programming visuals Various things that easily catch the eye priority This should get fixed as soon as possible! easy Good place to start if you're a new developer first-timers-only labels Aug 27, 2018
@DreadKnight DreadKnight added this to the 0.4 - Bounty Hunter milestone Aug 27, 2018
@johnnybigoode-zz
Copy link
Contributor

Hey, is anyone working on this? If not, I'd like to try! First Timer here.

@DreadKnight
Copy link
Member Author

DreadKnight commented Oct 2, 2018 via email

johnnybigoode-zz pushed a commit to johnnybigoode-zz/AncientBeast that referenced this issue Oct 3, 2018
@johnnybigoode-zz
Copy link
Contributor

You may find the change here: johnnybigoode-zz@2557632

Hey @DreadKnight ! After fiddling with the I found out that the offender was the bottom margin. After reading about Phaser, it seems that the default values are relevant for the correct rendering. So I managed to fix the issue by changing the default margin-bottom on styles.less:

image

The margin will still change size, but won't cut the canvas like before.

Is this valid? Maybe you could point me to more Phaser documentation, or some tips on where in the code should I look.

@DreadKnight
Copy link
Member Author

Great job, @johnnybigoode This did the trick, one less regression, woot! 👍

CyberBishop pushed a commit to CyberBishop/AncientBeast that referenced this issue Apr 20, 2023
CyberBishop pushed a commit to CyberBishop/AncientBeast that referenced this issue Apr 20, 2023
horizontal canvas rescale problems fixed, closes FreezingMoon#1444
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
coding This issue requires some programming easy Good place to start if you're a new developer priority This should get fixed as soon as possible! visuals Various things that easily catch the eye
Projects
None yet
Development

No branches or pull requests

2 participants