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.

Sign up for GitHub

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

Jump to bottom

"Next Phase" is confusing, consider changing the text as phases progress #11

Closed
uxmal opened this issue Jul 28, 2015 · 1 comment
Closed

"Next Phase" is confusing, consider changing the text as phases progress #11

uxmal opened this issue Jul 28, 2015 · 1 comment
Labels
enhancement This is a feature request

Comments

@uxmal
Copy link
Owner

uxmal commented Jul 28, 2015

The "Next Phase" button on the toolbar is confusing as it isn't clear what it will do. Consider changing the text of the button depending on what phase you're in.

To do this, the QueryStatus() mechanism needs to be expanded to accept a new text string. Suggestion: use a new enumeration, MenuStatus.SetText, which when set looks at the Text property of the MenuStatus and uses that to update the UI element.
@uxmal uxmal added the enhancement This is a feature request label Jul 29, 2015
uxmal added a commit that referenced this issue Jul 29, 2015
@uxmal
#11: Mouse Scroll will cause a crash
6c9e747
uxmal added a commit that referenced this issue Aug 28, 2015
@uxmal
#11 "Next Phase" button now changes text depending on phase to assist…
6fe00bd 
… beginners
@uxmal
Copy link
Owner Author

uxmal commented Aug 28, 2015

Implemented.

@uxmal uxmal closed this as completed Aug 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement This is a feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@uxmal