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

add a tab bar to the uilist showing categories, if there are any #77123

Merged
merged 2 commits into from
Oct 20, 2024

Conversation

db48x
Copy link
Contributor

@db48x db48x commented Oct 18, 2024

Summary

Interface "uilist menus that have categories now include a tabbar to allow the user to select a category"

Purpose of change

Describe the solution

Allows the user to select a category using the mouse, in addition to the existing key bindings.

Also tweaks how the size of the uilist is computed a bit, to keep the calculated_menu_size correct even when the caller specifies desired_bounds.

Testing

Start a game with Magiclysm, use the debug menu to edit your character, add some spells. Then check the spellcasting menu to see if it looks right. Then go back and add every spell (default key binding is t). Check the spellcasting menu again to make sure it still looks right.

Additional context

Screenshot from 2024-10-18 03-28-34
Screenshot from 2024-10-18 03-25-33

Allows the user to select a category using the mouse, in addition to
the existing key bindings.

Also tweaks how the size of the uilist is computed a bit, to keep the
calculated_menu_size correct even when the caller specifies desired_bounds.

Fixes CleverRaven#75666
Fixes CleverRaven#76746
@github-actions github-actions bot added Info / User Interface Game - player communication, menus, etc. [C++] Changes (can be) made in C++. Previously named `Code` <Bugfix> This is a fix for a bug (or closes open issue) labels Oct 18, 2024
@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Oct 18, 2024
This describes the purpose of the variable rather than how it was
obtained.
@github-actions github-actions bot added BasicBuildPassed This PR builds correctly, label assigned by github actions Items: Containers Things that hold other things labels Oct 19, 2024
@Maleclypse Maleclypse merged commit 435a445 into CleverRaven:master Oct 20, 2024
26 checks passed
@Brambor
Copy link
Contributor

Brambor commented Oct 21, 2024

As mentioned, this is likely the culprit of

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc. Items: Containers Things that hold other things json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Supernatural abilities menu, cutoff at the top Magiclysm spellcasting menu is missing its category selector
3 participants