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

Disabled mode for restart button #60

Merged
merged 2 commits into from Sep 24, 2020

Conversation

ahv15
Copy link
Member

@ahv15 ahv15 commented Sep 10, 2020

Contains

Part of Terasology/LightAndShadow#161 and adds the disabled mode to the UIButton which has to be used for the restart button.

How to test

Join a single player or multiplayer game and when the game is over, for the first 10 seconds the disabled mode of the refresh button will be visible.

Outstanding before merging

  • Disabled mode for UIButton.

Copy link
Contributor

@stefaniamak stefaniamak left a comment

Choose a reason for hiding this comment

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

Tested it out alongside Terasology/LightAndShadow#161 (review) and it works as expected 👍

I just want to suggest you making the letters slightly darker, so change the "E8E8E8" part of the color, with a darker grey HEX number :)

@stefaniamak
Copy link
Contributor

Also another note about making PRs: it's always nice to show a screenshot of your changes (especially when it is a UI change), so when reviewing we can be sure we saw what we expected to see!

@ahv15
Copy link
Member Author

ahv15 commented Sep 19, 2020

Screenshot (448)
@stefaniamak Is this color a bit too dark or is it fine?

@stefaniamak
Copy link
Contributor

I think it looks cool, yes!

@ahv15
Copy link
Member Author

ahv15 commented Sep 23, 2020

Done.

Copy link
Contributor

@stefaniamak stefaniamak left a comment

Choose a reason for hiding this comment

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

Tested it and works as expected 👍 I am merging it.

@stefaniamak stefaniamak merged commit 5cd9956 into Terasology:develop Sep 24, 2020
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.

None yet

2 participants