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

Adventure map, allow to delete the hero's predefined path on the adventure map regardless the state of the "Hero Movement" button #8924

Open
1 task done
LeHerosInconnu opened this issue Jul 6, 2024 · 2 comments · May be fixed by #9085
Labels
improvement New feature, request or improvement ui UI/GUI related stuff

Comments

@LeHerosInconnu
Copy link

Preliminary checks

Describe the problem requiring a solution

When the selected hero has movement points left, it is possible to delete the hero's predefined path on the adventure map by doing a left click held on the "Hero movement" button.
However, when the hero has no more movement points (the "Hero Movement" button is "grayed out"), it is no longer possible to do so.
I suggest that it should be possible to delete the hero's predefined path on the adventure map regardless of whether the "Hero Movement" button is "grayed out" or "not grayed out".

In fheroes2:

2024-07-06.19-03-52.mp4

Describe the possible solution

Allow to delete the hero's predefined path on the adventure map regardless the state of the "Hero Movement" button.

Additional info

No response

@LeHerosInconnu LeHerosInconnu added the improvement New feature, request or improvement label Jul 6, 2024
@zenseii zenseii added the ui UI/GUI related stuff label Jul 7, 2024
@zenseii zenseii added this to the Beyond OG scope milestone Jul 7, 2024
@AlexSnowLeo
Copy link
Contributor

Hi @LeHerosInconnu. This is the expected behavior, am I right?

fheroes2.engine.version_.1.1.1.2024-09-05.23-39-33.mp4

@LeHerosInconnu
Copy link
Author

Hello @AlexSnowLeo,

Hi @LeHerosInconnu. This is the expected behavior, am I right?

fheroes2.engine.version_.1.1.1.2024-09-05.23-39-33.mp4

Yes, exactly. :)

AlexSnowLeo added a commit to AlexSnowLeo/fheroes2 that referenced this issue Sep 5, 2024
to delete hero path always
@AlexSnowLeo AlexSnowLeo linked a pull request Sep 5, 2024 that will close this issue
@oleg-derevenetz oleg-derevenetz linked a pull request Sep 7, 2024 that will close this issue
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 a pull request may close this issue.

3 participants