Skip to content

Conversation

Usepot
Copy link
Contributor

@Usepot Usepot commented Jul 24, 2025

Adds metadata to the spawn packet to make the wither extremely small so the player doesn't see a wither outline when the bossbar is on low health aswell as stops the wither from dying if it has 0 "health" aka the bossbar isnt filled atall

Adds metadata to the spawn packet to make the wither extremely small so the player doesn't see a wither outline when the bossbar is on low health
@FlorianMichael
Copy link
Member

Could you provide a before/after screenshot maybe?

@Usepot Usepot changed the title Remove the Wither outline when using the bossbar on 1.9 -> 1.8 BossBar Wither Fixes 1.9 -> 1.8 Jul 24, 2025
@Usepot
Copy link
Contributor Author

Usepot commented Jul 24, 2025

when testing I found another bug where if the bossbar's "health" is 0 the wither on 1.8 will just die
{FED13B00-8F30-4CEC-88EC-D2DA10A18041} - Wither Having an outline at low health
{E10EC606-4F20-4C0B-A46E-2B249919937A} - Wither Having dying at 0 health
{0F4CB427-AF74-43B3-BC3B-C75CF4C47F2B} - Issue is not present on new version

@@ -201,7 +201,8 @@ private void addWither() {
entityData.add(new EntityData(0, EntityDataTypes1_8.BYTE, (byte) 0x20));
entityData.add(new EntityData(2, EntityDataTypes1_8.STRING, title));
entityData.add(new EntityData(3, EntityDataTypes1_8.BYTE, (byte) 1));
entityData.add(new EntityData(6, EntityDataTypes1_8.FLOAT, health * 300f));
entityData.add(new EntityData(6, EntityDataTypes1_8.FLOAT, Math.max(health * 300f, 1f))); // the wither will just die if the health is too low
Copy link
Member

Choose a reason for hiding this comment

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

Is this maybe related to the limit in #setHealth() above?

if (this.health <= 0) { this.health = 0.0001f; }

FlorianMichael pushed a commit that referenced this pull request Jul 27, 2025
Adds metadata to the spawn packet to make the wither tiny so the player doesn't see a wither outline when the bossbar is on low health
@FlorianMichael
Copy link
Member

Merged the first commit: af44f88

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