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

Pause menu enhancements #141

Merged
merged 30 commits into from
Apr 24, 2022

Conversation

pnwparksfan
Copy link
Contributor

@pnwparksfan pnwparksfan commented Mar 3, 2022

Several improvements to TabMissionSelectItem:

  • Added optional dynamic scaling of mission info area based on screen size and text length in item labels
  • Added dynamic resizing of mission logo texture to support various aspect ratios
  • Refactored Draw() function to render correctly as a submenu item
  • Updated examples to demonstrate these new capabilities
  • Fixed texture rendering when using Rage Graphics to render outside texture files (texture alignment was previously incorrect)

Improvements to TabSubmenuItem:

  • Dynamic scrolling when more items are available than fit on the screen
  • Fixed bug that occurs when an interactive menu or mission select menu is focused in a submenu, and the user navigates away using LB/RB. Previously the Focused property of the sub menu would remain true but the FocusLevel would be out of sync, making it impossible to back out of that submenu.

@pnwparksfan pnwparksfan force-pushed the pause-menu-enhancements branch from afc9c98 to 799dae1 Compare March 3, 2022 08:19
Source/PauseMenu/TabItem.cs Outdated Show resolved Hide resolved
Source/PauseMenu/TabMissionSelectItem.cs Outdated Show resolved Hide resolved
@thatmaxplayle
Copy link

thatmaxplayle commented Mar 9, 2022

As per PNWParksFan's request, I'm leaving this screenshot in the discussion for this PR.

  1. Having some weird issue with dynamic tab widths since using a binary with these enhancements.
  2. The text doesn't look brilliant when long strings are used. I don't think this is specific to this PR, more of an issue in general. But I have not tested it with the older version, so who knows... a dynamic title height (as PNW suggested) or maybe a dynamic text size for the title string?

image

@pnwparksfan
Copy link
Contributor Author

pnwparksfan commented Mar 12, 2022

@alexguirre do you think there should be different formatting for background color and foreground color on skipped items (in menus or tabs), like there is for disabled items on menus? And if so, should that formatting apply automatically or be optional? I added some default formatting to gray out skipped tabs in a TabView, but noticed that in a UIMenu a Skipped element isn't formatted different currently. It might be confusing to users to have a tab or menu item they can't select with no visual indicator.

271590_20220312001658_1

@pnwparksfan
Copy link
Contributor Author

pnwparksfan commented Mar 16, 2022

As per PNWParksFan's request, I'm leaving this screenshot in the discussion for this PR.

  1. Having some weird issue with dynamic tab widths since using a binary with these enhancements.
  2. The text doesn't look brilliant when long strings are used. I don't think this is specific to this PR, more of an issue in general. But I have not tested it with the older version, so who knows... a dynamic title height (as PNW suggested) or maybe a dynamic text size for the title string?

image

Item 2 resolved with d3976e3

@pnwparksfan pnwparksfan marked this pull request as ready for review April 8, 2022 08:33
@alexguirre alexguirre merged commit a5bb3b3 into alexguirre:master Apr 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants