[MC-2930] Heart bar doesn't show how much damage is dealt to the player

In versions prior to the singleplayer/multiplayer merge (before 1.3), the heart bar would briefly flash faint hearts depending on how many of them were lost when taking damage. This is gone since version 1.3, with no mention in the change logs. Here's a video comparison of the health bar between 1.2.5 and 1.4.2.

What I expected to happen was...
Blinking faint bright hearts showing how much health I had before taking damage. If I had 10 hearts, and I took 2 hearts of damage, I'd expect those 2 hearts to blink briefly before dissapearing completely.

What actually happened was...
No sign of how much hearts were lost. The hearts completely dissappear just after taking any damage.

Note:
This is not about regeneration, which works as expected. User Vincent Lee has found a relevant part of the code related to the bug.

Steps to reproduce:
1. Load a world on survival mode.
2. Take damage from any source.
3. Observe the heart bar. It should flash the hearts that were lost, but it does not.

Linked issues

is duplicated by 1

MC-4539 Textures not loaded after 1.2.5 Resolved

Attachments

Comments 27

Jack 2012-11-13T23:29:42Z

It actually flashes on the 2nd picture

CubeTheThird 2012-11-15T03:04:32Z

Based on your screenshots, I fail to see the problem here.

Daniel Garcia 2012-11-15T03:14:51Z

I'm uploading a video right now, but internet is slow at the moment. Expect a link in an hour or two.

Daniel Garcia 2012-11-15T04:50:47Z

Added example/comparison video.
I noticed YouTube messed up a little up with the video's "lighting", shouldn't be a problem though.

stephen locke 2012-11-15T05:02:29Z

It looks the same in 1.4.2 you are gaining health from your food, I see no problem.

17 more comments

Daniel Garcia 2012-11-15T05:05:57Z

It is not about the regeneration, regeneration works completely as expected.
It's about the bright hearts that blink when taking damage, which indicates how much health you had before taking damage. It is gone since Minecraft 1.3, with no mention in the change logs.

stephen locke 2012-11-15T05:24:38Z

Oh my mistake, I misread your complaint. You could be a little clearer in you description of the problem, but I see what you mean now.

Daniel Garcia 2012-11-15T10:40:04Z

I've made the description and summary a little clearer.

CubeTheThird 2012-11-15T13:15:07Z

Can confirm.

Tails 2013-03-15T17:55:08Z

Is this still a concern in the current Minecraft version? If so, please update the affected versions in order to best aid Mojang ensuring bugs are still valid in the latest releases/pre-releases.

insomniac_lemon 2013-08-07T18:17:54Z

Yes, still not fixed, even though they added new GUI icons that included new transition icons. Also, pack.png is once again broken.

Daniel Garcia 2013-09-12T04:56:23Z

Bug hasn't been fixed yet neither in latest release or snapshot at the time of writing.

williewillus 2013-09-18T20:17:06Z

Did some testing.
This bug has been in multiplayer ever since release 1.0 (it was fine in multiplayer beta 1.8)
With the conversion of singleplayer into multiplayer in 1.3 this bug was carried into singleplayer as well.

insomniac_lemon 2014-01-04T12:41:50Z

Still broken up to 1.7.4.

Itouch2 2014-02-09T17:27:02Z

Also affects 1.8 snapshots (06a/b)

Itouch2 2014-02-14T14:43:23Z

Still present in 07

Itouch2 2014-02-23T16:16:53Z

Still in 08a

Itouch2 2014-03-11T17:35:50Z

Confirmed for 1.7.5 and 14w10c

Itouch2 2014-03-15T21:28:16Z

Confirmed for 14w11b

Itouch2 2014-06-01T15:02:26Z

Confirmed for 14w21b.

Daniel Garcia 2014-06-01T20:24:56Z

I reworded almost everything in hopes to get attention more easily from Mojang. Sorry for making several small edits instead of just one large edit.

williewillus 2014-06-05T20:21:38Z

Update on bug hunting -
Turns out this problem can be solved by adding ONE LINE to the code.

Mojangsters: Using MCP names

In EntityClientPlayerMP 's attackEntityFrom() method.
Before the return false statement, insert this.prevHealth = getHealth() + par2;
this sets the correct clientsided prevHealth value so the gui can render the flashing hearts appropriately.

agentpaint 2014-07-31T03:41:13Z

I cannot confirm if this is still in 14w31a. I can't see the heart bar fast enough.

insomniac_lemon 2014-07-31T05:32:42Z

Confirmed for 14w31a.

williewillus 2014-08-07T22:24:21Z

Whatever Mojang tried in 14w32a/b does not work fully.

Test case:
Setup cactus. Hug cactus and stay there.
Expected outcome: Only the lost half heart increments flash each time.
Real outcome: All hearts lost from the cactus hug flash.
If you hug the cactus for ~10 secs about 5 of the hearts will flash.

This is incorrect as it does not show that 0.5 heart of damage was taken each time.

Please check and reopen.

Devs: Look at how it worked in 1.2.5. Replicate that.

Torabi 2014-08-07T23:31:20Z

That's part of the improvements. Did you watch the video demonstrating the new behavior?

insomniac_lemon 2014-08-08T01:34:09Z

I can see why it was done that way, because you are still receiving damage, this allows you to easily see how much damage you have received just before stopping the damage. For instance, if you are underwater with 9 hearts, if you start drowning you can clearly see you started with 9 and lost 3.

However, "saturation" hearts are the only heath that do not have these transition icons, and food points have the textures but they never worked in the first place.