search

Civcraft / Citadel

Do not open issues here; open them on the maintained fork @ DevotedMC
https://github.com/DevotedMC/Citadel
BSD 3-Clause "New" or "Revised" License
6 stars 23 forks source link

Add perc #175

Closed ProgrammerDan closed 8 years ago

ProgrammerDan commented 8 years ago

From @Vastitas -- allows optional display of health as a percent instead of just poor/well etc.

rourke750 commented 8 years ago

The current system is intentional. Is this configurable? On Mar 14, 2016 1:19 AM, "Daniel Boston" notifications@github.com wrote:

From @Vastitas https://github.com/Vastitas -- allows optional display

of health as a percent instead of just poor/well etc.

You can view, comment on, or merge this pull request online at:

https://github.com/Civcraft/Citadel/pull/175 Commit Summary

  • Adding ability to show health as percentage
  • Increment pom
  • Corrected import, method imprint

File Changes

Patch Links:

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/Citadel/pull/175.

Maxopoly commented 8 years ago

Could you change the default in the config and internally to false?

ProgrammerDan commented 8 years ago

Ah yeah, my bad, meant to change it to default "false".

@rourke750 Totally configurable. Easily disabled.

ProgrammerDan commented 8 years ago

I'll rebase.

Maxopoly commented 8 years ago

Do you want to rebase or should we just merge this?

ProgrammerDan commented 8 years ago

rebased. if build is good should be good to merge.