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

Revert "automatically open to favorites tab" #73237

Closed

Conversation

Vsevolod-Shustov
Copy link
Contributor

Summary

Interface "Revert 'Spellcasting menu - automatically open to favorites tab if player has favorited spells'"

Purpose of change

Make hotkeys for non-favorite spells work without extra left arrow press to go to "All" tab

Describe the solution

Make "All" tab default like it was since spellcasting menu was introduced

Describe alternatives you've considered

Make hotkeys for spells not in the current tab work *

  • I didn't actually consider coding it

Testing

None whatsoever

Additional context

#72643
#72849

…ayer has favorited spells"

UX is just bad, even author agree with it.
CleverRaven#72643
@github-actions github-actions bot added Info / User Interface Game - player communication, menus, etc. [C++] Changes (can be) made in C++. Previously named `Code` labels Apr 24, 2024
@github-actions github-actions bot requested a review from KorGgenT April 24, 2024 15:15
@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 Apr 24, 2024
@Brambor
Copy link
Contributor

Brambor commented Apr 28, 2024

@Blueflowerss might want to comment?

@Vsevolod-Shustov
Copy link
Contributor Author

I'll bump this as @Blueflowerss doesn't seem interested in commenting.

@I-am-Erk
Copy link
Member

What's the reason for this? Neither this nor your issue explain the problem, so I have no way to tell if we should be merging this or not.

@Vsevolod-Shustov
Copy link
Contributor Author

Vsevolod-Shustov commented May 11, 2024

What's the reason for this? Neither this nor your issue explain the problem, so I have no way to tell if we should be merging this or not.

Spell hotkeys only work when the spell is in the currently displayed list (all, favorites, magic/psionic schools). Before the PR I want reverted it wasn't an issue as by default spellcasting menu displayed "all" list, thus to cast a spell you assigned to a hotkey, you pressed spellcasting hotkey, then spell hotkey (two buttons). After that pr (now) spellcasting hotkey opens "favorites" list, and to cast arbitrary spell you press spellcasting hotkey, left arrow (to switch to "all" list), spell hotkey (three buttons).

@Fris0uman
Copy link
Contributor

We should probably fix the hotkey behaviour instead and make it work on any tab, no?

@Vsevolod-Shustov
Copy link
Contributor Author

Vsevolod-Shustov commented May 30, 2024

Can UX regression and broken muscle memory be fixed before that?

@Brambor
Copy link
Contributor

Brambor commented May 30, 2024

You could also add an option for what default tab to open. But CDDA devs don't like too many settings.

Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not bump or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered.

@github-actions github-actions bot added the stale Closed for lack of activity, but still valid. label Jun 29, 2024
@Brambor
Copy link
Contributor

Brambor commented Jun 30, 2024

I am closing this since the purpose was solved by:

@Brambor Brambor closed this Jun 30, 2024
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 [C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc. json-styled JSON lint passed, label assigned by github actions stale Closed for lack of activity, but still valid.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants