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

Support for Step Statistics in Ultrawide Double #351

Draft
wants to merge 1 commit into
base: beta
Choose a base branch
from

Conversation

Roujo
Copy link
Contributor

@Roujo Roujo commented Dec 28, 2021

Lots of values are arbitrary and hardcoded, but this looks pretty okay.

image

What's missing is QA and code review. The test cases I'll have to go through are combinations of the following:

Resolutions:

  • Ultrawide
  • 16:9
  • 4:3

Styles:

  • P1 Single
  • P1 Centered Single
  • P1 Double
  • P2 Single
  • P2 Centered Single
  • P2 Double
  • P1 + P2 (Versus)

Game Modes:

  • Pump It Up Support, since the notefield is larger. Do we want to support and test this as well?

@teejusb
Copy link
Collaborator

teejusb commented Jan 2, 2022

I should also mention that I added a scorebox to step stats in #345 . Can you verify that the positioning/sizing is still correct (you'll need to use the launcher to check this)?

@natano natano added the enhancement Improve existing functionality label Jan 8, 2022
@Roujo
Copy link
Contributor Author

Roujo commented Jan 27, 2022

I should also mention that I added a scorebox to step stats in #345 . Can you verify that the positioning/sizing is still correct (you'll need to use the launcher to check this)?

Yup, will do! ^^

@teejusb
Copy link
Collaborator

teejusb commented Dec 13, 2023

Is this PR still in progress or should I close?

@Roujo
Copy link
Contributor Author

Roujo commented Dec 13, 2023

I haven't looked into doing the required QA in a while, but it is a feature I'd like to see in SL so I'll put that back on my plate, thank you for the reminder! 😄

(Wow, it's been 2 years, incredible 🤣)

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

Successfully merging this pull request may close these issues.

3 participants