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

[QoL] Auto center the window vertically #2686

Merged

Conversation

Dakurei
Copy link
Collaborator

@Dakurei Dakurei commented Jun 28, 2024

What are the changes?

Auto center the window vertically when the game is played in landscape format or in portrait format when touch controls are not displayed

Why am I doing these changes?

It was a request, and an old PR closed in advance in favor of this one (See #1114) (cc @torranx)

What did change?

CSS modification to achieve alignment without having to tweak any config in the game code directly

Screenshots/Videos

Before
2024-06-28_19-19
2024-06-28_19-19_1

After
2024-06-28_19-15
2024-06-28_19-16

centered-vertical

How to test the changes?

Launch from a mobile or using an adaptive browser view

Checklist

  • There is no overlap with another PR?
  • The PR is self-contained and cannot be split into smaller PRs?
  • Have I provided a clear explanation of the changes?
  • Have I tested the changes (manually)?
    • Are all unit tests still passing? (npm run test)
  • Are the changes visual?
    • Have I provided screenshots/videos of the changes?

@Dakurei Dakurei added Enhancement New feature or request UI/UX User interface/-experience related labels Jun 28, 2024
torranx
torranx previously approved these changes Jun 29, 2024
Copy link
Contributor

@torranx torranx left a comment

Choose a reason for hiding this comment

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

Lgtm. tested and no particular errors encountered

@Dakurei Dakurei force-pushed the always-centered-landscape branch from 3272896 to 0e4c78a Compare July 2, 2024 18:05
@Dakurei Dakurei force-pushed the always-centered-landscape branch from 0e4c78a to 7a01282 Compare July 10, 2024 17:31
@Dakurei Dakurei changed the base branch from main to beta July 10, 2024 17:40
@Dakurei Dakurei force-pushed the always-centered-landscape branch from 7a01282 to b5d9cdd Compare July 25, 2024 12:10
@Dakurei Dakurei force-pushed the always-centered-landscape branch from b5d9cdd to 2bcf0de Compare July 28, 2024 21:41
@torranx torranx requested a review from a team August 31, 2024 19:10
@torranx torranx added the Lead Dev Review Lead Dev Review is required label Aug 31, 2024
  + When the game is played in landscape format
    or in portrait format when it's not a touchscreen
@Dakurei Dakurei force-pushed the always-centered-landscape branch from 2bcf0de to 88d5dee Compare September 3, 2024 15:20
@flx-sta
Copy link
Contributor

flx-sta commented Sep 17, 2024

@Dakurei Are you still actively working on this?

@f-fsantos f-fsantos merged commit 1fb5389 into pagefaultgames:beta Sep 17, 2024
4 checks passed
@Dakurei
Copy link
Collaborator Author

Dakurei commented Sep 17, 2024

@Dakurei Are you still actively working on this?

The fact is, it was all over the moment I created the PR xD

I was just doing the occasional rebase & force push when I came across the tab and saw conflicts, but since you've passed the option to always be in sync with Beta's HEAD, I wasn't going to spend my life rebasing 😂

@Dakurei Dakurei deleted the always-centered-landscape branch September 17, 2024 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request Lead Dev Review Lead Dev Review is required UI/UX User interface/-experience related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants