Skip to content

Group input refactor#389

Merged
Gunoshozo merged 1 commit intomasterfrom
group-input-refactor
Feb 19, 2026
Merged

Group input refactor#389
Gunoshozo merged 1 commit intomasterfrom
group-input-refactor

Conversation

@Gunoshozo
Copy link
Copy Markdown
Contributor

Group Input Refactor

  • Decouple Update and UpdateInput for Group
  • Add separate UpdateInput for relevant groups
  • Fix CHLCC Music Menu out of bounds hover
  • Use Enum state instead of boolean

@Gunoshozo Gunoshozo force-pushed the group-input-refactor branch from 66759a7 to 5a21bcf Compare February 14, 2026 08:15
@Gunoshozo Gunoshozo marked this pull request as ready for review February 14, 2026 09:28
@Gunoshozo Gunoshozo force-pushed the group-input-refactor branch from 5a21bcf to a3d9002 Compare February 14, 2026 11:33
Copy link
Copy Markdown
Member

@PringlesGang PringlesGang left a comment

Choose a reason for hiding this comment

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

Groups that still need to get an UpdateInput call:

  • src/games/darling/sysmesbox.cpp ChoiceItems
  • src/games/mo6tw/sysmesbox.cpp ChoiceItems
  • src/games/mo6tw/titlemenu.cpp ContinueItems
  • src/games/mo6tw/titlemenu.cpp SystemItems
  • src/games/mo6tw/titlemenu.cpp ExtraStoryItems
  • src/games/mo6tw/titlemenu.cpp MemoriesItems

Comment thread src/ui/widgets/group.h Outdated
@Gunoshozo Gunoshozo force-pushed the group-input-refactor branch from 08390d9 to e9eba08 Compare February 15, 2026 20:13
@Gunoshozo Gunoshozo requested a review from KKhanhH February 15, 2026 20:16
Comment thread src/games/cclcc/librarymenu.cpp Outdated
ScrWork[SW_SYSSUBMENUCT] == 0) {
State = Hidden;
MainItems.IsShown = false;
MainItems.VisibilityState = Shown;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this should be hidden

}
MainItems.Hide();
MainItems.IsShown = true;
MainItems.VisibilityState = Shown;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is technically Hiding state but we don't really have anything for that rn

@Gunoshozo Gunoshozo self-assigned this Feb 19, 2026
- Decouple Update and UpdateInput for Group
- Add separate UpdateInput call for relevant groups
- Fix CHLCC Music Menu out of bounds hover
- Use Enum state instead of boolean
@Gunoshozo Gunoshozo force-pushed the group-input-refactor branch from 1ed044f to d3de3fd Compare February 19, 2026 19:17
@Gunoshozo Gunoshozo merged commit e3985b6 into master Feb 19, 2026
11 checks passed
@Gunoshozo Gunoshozo deleted the group-input-refactor branch February 19, 2026 20:30
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.

Out of bounds Group items can request a pointer cursor and are hoverable

3 participants