ViaVersion / ViaFabricPlus

Fabric mod to connect to EVERY Minecraft server version (Release, Beta, Alpha, Classic, Snapshots, Bedrock) with QoL fixes to the gameplay
GNU General Public License v3.0
420 stars 44 forks source link

Glass Panes still have leftover pixels #558

Closed christopherplaysminecraft closed 2 months ago

christopherplaysminecraft commented 2 months ago

General Information

VFP 3.4.5

Bug Description

ViaFabricPlus changes the hitbox and block outline of glass panes, but doesn't remove the then-invalid pixels. 1.8.x

Block-Outline-Selected-Glass-Pane Block-Outline-Unselected-Glass-Pane

Steps to Reproduce

Look at two connected glass panes.

Expected Behavior

ViaFabricPlus shouldn't be rendering those pixels.

FlorianMichael commented 2 months ago

Visual only changes in general aren't ideal for ViaFabricPlus since they usually only make the mod even more hard to maintain (given how big the code base already is), and especially since this does not affect game play at all and is (in my opinion) barely noticable, I'll close this as won't fix.

We can do visual changes, but only if they are appropriate in terms of code size and also useful. Some examples for this would be:

christopherplaysminecraft commented 2 months ago

Is the issue with maintaining or mod incompatibilities?

FlorianMichael commented 2 months ago

It's just not worth at all because no one would really care and it would just introduce new code which would need updating any time we update to a newer Minecraft version.

christopherplaysminecraft commented 2 months ago

I have had a surprising number of people comment on this actually, mostly people playing Hypixel Skyblock, in which glass and glass panes are used to represent "gemstones"

Any chance it could be implemented as off-by-default, with no one checking if it works on the latest version, and only fixing it if someone reports it being broken? (Or would the code being broken stop VFP from compiling at all?)

Felix14-v2 commented 2 months ago

Afaik you can just use a resourcepack with fixed models to hide the "rod" of the panel.

especially since this does not affect game play at all

Do I understand correctly that the intangible part of the panel does not exist for the server? I'm just worried about anticheats.

FlorianMichael commented 2 months ago

ViaFabricPlus changes the bounding box/collision but not the rendering causing the block to look "wrong"

christopherplaysminecraft commented 2 months ago

I thought about it for a bit. It would take 5-20 hours depending on ones skill level to make this work, as one would also most likely have to add extra models to be used when Continuitys Glass Pane Culling Fix Resource Pack is enabled.