-
Notifications
You must be signed in to change notification settings - Fork 252
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
Enhance mission levels select window #613
Conversation
f9b2543
to
034d9da
Compare
Nice work, it looks great! Tested in Windows. Do you mind if the core missions go to the top of the list? See here for reference. I also really like the idea of the |
034d9da
to
ca26865
Compare
I'm planning to rewrite "New Game" UI flow and divide Campaign and Mission select to different screens. Regarding "Level select" title I'll add name of mission. |
ca26865
to
8f8bcb4
Compare
Changed "Select Level" with mission name. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love this new feature! Tested with success on Linux. I also have a request, not necessarily for this MR: it could be great to display the number of levels of a mission in the "info" dialog below the "notes" header when selecting the mission.
8f8bcb4
to
a976759
Compare
This function may be used for fast information accessing such as level name or description.
Now level loading screen shows level name, and user can choose level from list instead of manually entering level number.
a976759
to
83886a2
Compare
That would be kinda spoiler for player, he will know how many missions are left. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested again on Linux, I think we're good now!
Pull Request Type
Description
This PR was inspired by #485. Now missions that contains more that one level, shows enhanced level select window (see screenshot). This enhancement allows user to select available level instead of input number of mission manually. Additional refactoring made for filename paths used in related functions.
Related Issues
Screenshots (if applicable)
Checklist
Additional Comments