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

Game board side-bar updates. #114

Closed
8 tasks done
lodici opened this issue Mar 21, 2015 · 8 comments
Closed
8 tasks done

Game board side-bar updates. #114

lodici opened this issue Mar 21, 2015 · 8 comments
Assignees
Milestone

Comments

@lodici
Copy link
Member

lodici commented Mar 21, 2015

screenshot00059

  • merge user-action panel with player panel.
  • improve signal-to-noise ratio in user-action panel by restricting to user-decision prompts only.
  • suppress user-action panel if playing an AI vs AI game.
  • position of the four side-bar components should be configurable.
  • remove priority player from turn status panel. Inconsistent.
  • display overall duel score in turn status panel.
  • display duel target as tooltip (eg. "First player to win 4 games wins the duel.").
  • display number of items in stack.
@lodici lodici added this to the 1.60 milestone Mar 21, 2015
@lodici lodici self-assigned this Mar 21, 2015
@lodici
Copy link
Member Author

lodici commented Mar 22, 2015

You can now re-arrange the components in the duel side-bar. No UI as yet but you can do it manually by adding or updating the following property in \Magarena\general.cfg -

duelSidebarLayout=LOGSTACK,PLAYER2,TURNINFO,PLAYER1

Simply move the order around to have it reflected in the display (restart required).

I have also made the above order the default order -
screenshot00071

@lodici
Copy link
Member Author

lodici commented Mar 26, 2015

Added dialog to arrange side-bar components
duelsidebarlayoutdialog

@ShawnieBoy
Copy link
Member

Nice

@lodici
Copy link
Member Author

lodici commented Mar 29, 2015

I have added a new setting to the gameplay preferences tab. Off by default, if enabled it will suppress the prompts in the user action panel unless a decision is required by you. I find this a big improvement on the signal-to-noise ratio not least of which is the removal of the prompt that tells you what the AI is going to play about 5 seconds before it plays it..

I could not come up with a particularly elegant solution, requiring lots of &&ing and ||ing, (see isUserActionPromptHidden in SwingGameController) so if you know of a better way please update.

I have played quite extensively with this setting enabled and so far it seems to be working as expected but there is no substitute for more people testing so please give it a go and let me know what you think. My intention is to release it ON by default in 1.61 and if there are no major objections remove it as a setting in 1.62.

@lodici lodici closed this as completed Mar 29, 2015
@melvinzhang
Copy link
Contributor

The reason AI message are shown is due to the showMessage call in getArtificialNextEventChoiceResults. I've added a check with getHideAiActionPrompt before calling showMessage.

@melvinzhang
Copy link
Contributor

Btw, why does clearUserAction prompt have the condition (game.getPhase().getType() != MagicPhaseType.DeclareAttackers) ?

@melvinzhang
Copy link
Contributor

I think I see it, you have to avoid DeclareAttackers because otherwise after choosing the first attacker the message is removed. I think the MagicChoices were not designed with the message being cleared after the first response. We can get the same effect without the awkward check by clearing the user action prompt when it is the AI's decision. Implemented in e23debb

@melvinzhang
Copy link
Contributor

The final implementation is simpler: if hide AI choices is true, during AI's choice show an empty message to clear the user action prompt, otherwise show the actual message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants