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

Interface setting in "Evil" or "Good" mode is not retained when the user goes to the Home screen and returns to the game #2450

Closed
LeHerosInconnu opened this issue Dec 20, 2020 · 12 comments
Labels
low priority Low priority item which could be addressed in the future question Question asked ui UI/GUI related stuff
Milestone

Comments

@LeHerosInconnu
Copy link

In fheroes2, the interface setting in "Evil" mode or "Good" mode is not retained when the user goes to the Home screen and returns to the game.
In the example, when the user returns from the Home screen, the interface should be "Good".

In fheroes2:

2020-12-20.11-13-28.mp4
@Branikolog Branikolog added the question Question asked label Dec 20, 2020
@Branikolog
Copy link
Collaborator

Hi, @LeHerosInconnu
Couldn't reproduce your issue in such way. I found out that the game always starts with good interface mode until you've set in CONFIG menu the "Evil" interface. And no matter, what kind of interface you've chosen in game System Options, It will always run according to setting in CONFIG menu.
Are you sure, you haven't set "Evil" in config in this example?

@LeHerosInconnu
Copy link
Author

@Branikolog.

Hi, @LeHerosInconnu
Couldn't reproduce your issue in such way. I found out that the game always starts with good interface mode until you've set in CONFIG menu the "Evil" interface. And no matter, what kind of interface you've chosen in game System Options, It will always run according to setting in CONFIG menu.
Are you sure, you haven't set "Evil" in config in this example?

Not sure about that.
I'll do some tests later.

But in any case, if the user makes changes anywhere in the interface, they must be taken into account.

Also I think that the option "game: use evil interface" should be removed from the options in the "Experimental Game Settings" window.

@Branikolog
Copy link
Collaborator

@LeHerosInconnu.
For the case, when player makes changes and saves the game - changes in a savegame should be applied after loading. I'm agreed.
But what about the case in your video? You've started a new game. Should we save interface type? New Game always starts on "good" interface... Is it ok for the case, when player start new game, he start with some non-default options applied... New game = default options?

@LeHerosInconnu
Copy link
Author

@Branikolog.

@LeHerosInconnu.
For the case, when player makes changes and saves the game - changes in a savegame should be applied after loading. I'm agreed.
But what about the case in your video? You've started a new game. Should we save interface type? New Game always starts on "good" interface... Is it ok for the case, when player start new game, he start with some non-default options applied... New game = default options?

Changes made by the player should always be saved, even the next time the program is started.
It works this way in the original game.

@LeHerosInconnu
Copy link
Author

@Branikolog.

It's heavily broken. 👎

In fheroes2:

2020-12-20.15-43-19.mp4

@Branikolog
Copy link
Collaborator

@LeHerosInconnu.
Yes, @ihhub already knows about this issue with CONFIG.
Cannot say, it's sooo "heavily". ))

@Branikolog Branikolog added low priority Low priority item which could be addressed in the future ui UI/GUI related stuff labels Dec 22, 2020
@Branikolog Branikolog added this to the 0.9.1 milestone Dec 22, 2020
@Branikolog
Copy link
Collaborator

I've checked the OG. It works simply there: once you've changed GUI "Evilness", it will be evil everywhere. Each time you load, or start new game. That's true.
In my personal point, I'd prefer to save a type of GUI into a savefile, so if I play one scenario in evil form, it should be loaded with evil GUI. But when I load another scenario, for which I had chosen a "Good" GUI, the game should remember that and show "Good" GUI...
But currently, interface works incorrectly, comparing to OG.

@LeHerosInconnu
Copy link
Author

@Branikolog.

...so if I play one scenario in evil form, it should be loaded with evil GUI. But when I load another scenario, for which I had chosen a "Good" GUI, the game should remember that and show "Good" GUI...

In fheroes2, the "Interface Dynamic" setting of the original game is missing: #1329. :)

@Branikolog
Copy link
Collaborator

Branikolog commented Dec 22, 2020

@LeHerosInconnu
Isn't the dynamic interface just chooses the default type for playing game, according to the starting race selected?

@LeHerosInconnu
Copy link
Author

@Branikolog.

@LeHerosInconnu
Isn't the dynamic interface just chooses the default type for playing game, according to the starting race selected?

If you start a scenario with the dynamic interface, the game will select the interface accordingly.
If you change the interface during the scenario, the program will remember it the next time the game loads.

@ihhub ihhub modified the milestones: 0.9.1, 0.9.2 Mar 5, 2021
ihhub added a commit that referenced this issue Mar 15, 2021
ihhub added a commit that referenced this issue Mar 15, 2021
@Branikolog Branikolog modified the milestones: 0.9.2, 0.9.3 Apr 6, 2021
@ihhub ihhub modified the milestones: 0.9.3, 0.9.4 May 5, 2021
@ihhub ihhub modified the milestones: 0.9.4, 0.9.7 Jun 3, 2021
@ihhub ihhub modified the milestones: 0.9.7, 0.9.8 Sep 5, 2021
@ihhub ihhub modified the milestones: 0.9.8, 0.9.9 Oct 5, 2021
@ihhub ihhub removed this from the 0.9.9 milestone Nov 5, 2021
@ihhub ihhub added this to the 0.9.10 milestone Nov 5, 2021
@ihhub ihhub modified the milestones: 0.9.10, 0.9.11 Dec 5, 2021
@ihhub ihhub modified the milestones: 0.9.11, 0.9.12 Dec 23, 2021
@ihhub ihhub modified the milestones: 0.9.12, 0.9.13 Feb 7, 2022
@ihhub ihhub modified the milestones: 0.9.13, 0.9.14 Mar 7, 2022
@ihhub ihhub modified the milestones: 0.9.14, 0.9.15 Apr 9, 2022
@zenseii
Copy link
Collaborator

zenseii commented Apr 14, 2022

Did the PR #5272 solve this? It seems like this and #5251 were duplicate issues, although some other points are raised in later comments here, like interface type being saved for every save file. Maybe the issue title should be changed or the issue should be closed @LeHerosInconnu?

@LeHerosInconnu
Copy link
Author

Hello @zenseii,

The problem described in the original post seems to be solved.
I will do a test of functioning in the original game later and open another issue if necessary.
Thanks for your comment.
I close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low priority Low priority item which could be addressed in the future question Question asked ui UI/GUI related stuff
Projects
None yet
Development

No branches or pull requests

4 participants