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

New Player HUD #2442

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

New Player HUD #2442

wants to merge 7 commits into from

Conversation

bucketh3ad
Copy link
Member

Simplifies the HUD

Basic icon number for health and money
Weapon icon and ammo amount
Flashing icons for potion effects
Badge for the active quest.

TODO:

  • Add armor icons

@bucketh3ad bucketh3ad added the wip label Jun 12, 2015
@edisonout
Copy link
Member

I don't like potion icons flashing - if you use another one they start flashing out of sync. I think it would look better as just an image.

The saving icon pops up over the weapon projectile amount. Not a fan of magnitude, but if other people want to keep him, he can stay.

I think the weapon amount value is too far to the right - there's a bigger space between the picture & the text than there is for money or health. Do we want to keep the "x"?

Not sure the effects are working properly - when I open the game with all cheats on, I have green, blue & orange potions displayed. If I use all the potions then at the end when they've finished I have green and purple instead. I'm not sure having all these different icons is helpful to the player but I don't know what else to suggest.

@8bitgentleman
Copy link
Contributor

I agree about the flashing, I don't think it looks good.

@8bitgentleman
Copy link
Contributor

Is it possible to sort this and the enemy hud out for this weekend's bug fix release?

@niamu niamu force-pushed the master branch 2 times, most recently from de2bca9 to 20307b1 Compare September 25, 2015 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants