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

V0.97 post sprint 4 fixes/updates #481

Merged
merged 3 commits into from
Aug 26, 2018
Merged

V0.97 post sprint 4 fixes/updates #481

merged 3 commits into from
Aug 26, 2018

Conversation

Wyqer
Copy link
Member

@Wyqer Wyqer commented Aug 25, 2018

Q A
Bug fix? yes
New feature? no
Needs wipe? no
Fixed issues #471

Description:

Small fixes and updates for the sprint 4 result.
Note: Currently we can't fix the "directly close bug" for the admin menu as it only contains buttons. Later I guess we'll add a small "statistics/info area" at the top of the dialog and make sure one of the dynamic text controls will be the first focus.

Content:

  • Fixed a typo for the cam close function in the build module
  • Updated the player menu to the state of the standalone version
  • Fixed direct close in player menu

Tested on:

  • Local MP Vanilla
  • Local MP ACE
  • Dedicated MP Vanilla
  • Dedicated MP ACE

@Wyqer Wyqer added this to the Sprint 5 milestone Aug 25, 2018
@Wyqer Wyqer requested a review from veteran29 August 25, 2018 21:31
@Wyqer Wyqer changed the title V0.97 s5 471 V0.97 post sprint 4 fixes/updates Aug 25, 2018
Copy link
Member

@veteran29 veteran29 left a comment

Choose a reason for hiding this comment

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

lgtm

@Wyqer Wyqer merged commit 28994b0 into v0.97S5 Aug 26, 2018
@Wyqer Wyqer deleted the v0.97S5-471 branch August 26, 2018 09:43
@Wyqer Wyqer added done and removed review/QA labels Aug 26, 2018
@Wyqer Wyqer mentioned this pull request Sep 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants