add mouse support to mission menu (M) #60084
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
[C++]
github-actions
bot
added
astyled
Hirmuolio
pushed a commit
to Hirmuolio/Cataclysm-DDA
that referenced
this pull request
Aug 27, 2022
* add mouse support to mussuin menu * fix astyle Co-authored-by: Тимур Кашафутдинов <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Features "Mission menu window (M) now has mouse support: you can click on tabs, hover over mission names and click on mission names to make it active"
Purpose of change
fixes #60083
expand mouse support to yet another UI window
Describe the solution
Describe alternatives you've considered
Testing
Generate lots of missions
Open mission menu
Make sure you can switch tabs by clicking on tab name
Make sure hovering on mission name row show you mission description
Make sure clicking on mission name on active mission tab makes this mission active
Make sure mouse scroll scroll missions up and down
Additional context
Im not quite happy with the result, it could use some optimization. But it will do as MVP