Skip to content

Improved UI in the Scenario Selection dialogue screen - #10808

Draft
AchFilip wants to merge 7 commits into
ihhub:masterfrom
AchFilip:improvement/10787-back-button-in-load-map-menu
Draft

AchFilip wants to merge 7 commits into
ihhub:masterfrom
AchFilip:improvement/10787-back-button-in-load-map-menu

Conversation

@AchFilip

@AchFilip AchFilip commented Jun 2, 2026

Copy link
Copy Markdown
Contributor
  • Added Okay & Cancel buttons w/ left and right click triggers
  • Hiding the Okay button that exists in the image.

Below you can see a video proof.

601745586-01ab52db-e6e8-460b-b525-4095973eb575.mp4

Related issue #10787

AchFilip added 2 commits June 1, 2026 18:55
- Masking the okay button that exists in the map selection image
@ihhub ihhub added improvement New feature, request or improvement ui UI/GUI related stuff labels Jun 3, 2026
@ihhub ihhub added this to the 1.1.17 milestone Jun 3, 2026
@Branikolog

Copy link
Copy Markdown
Collaborator

Hi, @AchFilip !
I haven't tested the build itself since it hasn't been compiled yet. However, from what I could see in the video, both new buttons appear to be missing their shadows. If this is still relevant, could you please add them as well?

@AchFilip

AchFilip commented Jun 3, 2026

Copy link
Copy Markdown
Contributor Author

Hi, @AchFilip ! I haven't tested the build itself since it hasn't been compiled yet. However, from what I could see in the video, both new buttons appear to be missing their shadows. If this is still relevant, could you please add them as well?

Hey @Branikolog,

Yeah I'll fix the code styling first and then I'll check the shadow thingy.
Missed that completely, my bad!

- Fixed issue where having different language will result in wrong alignment for the cancel button
- Rendering shadow for both buttons

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Clang-Tidy found issue(s) with the introduced code (1/1)

Comment thread src/fheroes2/dialog/dialog_selectscenario.cpp Outdated
AchFilip and others added 4 commits June 3, 2026 23:54
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@ihhub

ihhub commented Jun 4, 2026

Copy link
Copy Markdown
Owner

Hi @AchFilip ,

This particular window should be completely reworked as we want it to be vertically resizable and be wider due to very little space left for map names. I know that this is a huge work to be done but we could fix many issues by doing this.

If you keen to give it a try I recommend to take a look at #10573 how other windows were done and also at dialog_resolution.cpp file (more specifically applyTextBackgroundShading() method which is helpful to replicate the needed shapes).

I understand that this is a very tedious task but it would also be very rewarding for players as this dialog is used not only for selection of maps in the Main Menu but also in the Editor. Moreover, me and @Districh-ru can help you with this if needed.

What do you think?

@AchFilip

AchFilip commented Jun 4, 2026

Copy link
Copy Markdown
Contributor Author

Hi @AchFilip ,

This particular window should be completely reworked as we want it to be vertically resizable and be wider due to very little space left for map names. I know that this is a huge work to be done but we could fix many issues by doing this.

If you keen to give it a try I recommend to take a look at #10573 how other windows were done and also at dialog_resolution.cpp file (more specifically applyTextBackgroundShading() method which is helpful to replicate the needed shapes).

I understand that this is a very tedious task but it would also be very rewarding for players as this dialog is used not only for selection of maps in the Main Menu but also in the Editor. Moreover, me and @Districh-ru can help you with this if needed.

What do you think?

Hey @ihhub !

I totally agree.
One more issue the current implementation has is it does not change theme between evil/good.
Yeah for sure! I'm totally hooked-up with this project.

@ihhub

ihhub commented Jun 4, 2026

Copy link
Copy Markdown
Owner

Hi @AchFilip ,
This particular window should be completely reworked as we want it to be vertically resizable and be wider due to very little space left for map names. I know that this is a huge work to be done but we could fix many issues by doing this.
If you keen to give it a try I recommend to take a look at #10573 how other windows were done and also at dialog_resolution.cpp file (more specifically applyTextBackgroundShading() method which is helpful to replicate the needed shapes).
I understand that this is a very tedious task but it would also be very rewarding for players as this dialog is used not only for selection of maps in the Main Menu but also in the Editor. Moreover, me and @Districh-ru can help you with this if needed.
What do you think?

Hey @ihhub !

I totally agree. One more issue the current implementation has is it does not change theme between evil/good. Yeah for sure! I'm totally hooked-up with this project.

Thank you! Please feel free to ask aby questions.

@AchFilip
AchFilip marked this pull request as draft June 8, 2026 10:32
@ihhub ihhub modified the milestones: 1.1.17, 1.1.18 Jun 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement New feature, request or improvement ui UI/GUI related stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants