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

feat: add water droplet icon beside the Thirst Bar #29

Merged
merged 6 commits into from
Oct 21, 2021
Merged

feat: add water droplet icon beside the Thirst Bar #29

merged 6 commits into from
Oct 21, 2021

Conversation

mohamedation
Copy link
Contributor

add water droplet icon indicator on the left of the thirst bar
Screenshot from 2021-10-13 14-53-19

Copy link
Member

@jdrueckert jdrueckert left a comment

Choose a reason for hiding this comment

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

I feel like it helps associating the contents of this file with what you can see in-game if the elements are in the same "order", aka if the icon comes first and then the bar.

Further, I think we should make the horizontal placement of the bar relative to the icon. Thus, they'll always have the same offset even if you decide to move it further right or left by adjusting the icon's horizontal placement. I would recommend the following for the bar:

"position-left": {
    "target": "LEFT",
    "widget": "thirstIcon",
    "offset": 45
},

assets/ui/Thirst.ui Outdated Show resolved Hide resolved
assets/ui/Thirst.ui Outdated Show resolved Hide resolved
mohamedation and others added 4 commits October 16, 2021 12:47
Co-authored-by: jdrueckert <jd.rueckert@googlemail.com>
Co-authored-by: jdrueckert <jd.rueckert@googlemail.com>
@mohamedation
Copy link
Contributor Author

thank you. all have been revised as suggested.

Copy link
Member

@jdrueckert jdrueckert left a comment

Choose a reason for hiding this comment

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

Basically same story as for the hunger PR 😉

@mohamedation
Copy link
Contributor Author

@jdrueckert jdrueckert merged commit bdccb3e into Terasology:develop Oct 21, 2021
@jdrueckert
Copy link
Member

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