Skip to content

Display Link: Value Display Options #995

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

Closed
Damdrara opened this issue Jul 7, 2023 · 5 comments
Closed

Display Link: Value Display Options #995

Damdrara opened this issue Jul 7, 2023 · 5 comments
Labels
type: bug Issue where something isn't working

Comments

@Damdrara
Copy link

Damdrara commented Jul 7, 2023

Describe the Bug

When selecting "List matching Items", the Value display option on the Fabric version gives the following options:

image

...which doesn't make a whole lot of sense for items.

Reproduction Steps

  1. Put a content observer / smart observer on a vault
  2. put a display link on the observer
  3. check the options

Expected Result

The forge version gives these:

image

Screenshots and Videos

No response

Crash Report or Log

No response

Operating System

Windows 11

Mod Version

0.5.1b

Minecraft Version

1.19.2

Other Mods

No response

Additional Context

also happens on 0.5.0i

@Damdrara Damdrara added the type: bug Issue where something isn't working label Jul 7, 2023
@github-actions github-actions bot added the 1.19 label Jul 7, 2023
@electricsteve
Copy link

first, make sure it is pointing at a container with items in it, second, the forge screenshot is taken in older version, so maybe it isn't rellevent in the newest version.

@Damdrara
Copy link
Author

Damdrara commented Jul 7, 2023

It definitely has items in it, that's not the issue.

I tested the bug on various versions, with the exact same world. it occures on Fabric on both 0.5.0i with the Content Observer and 0.5.1b with the Smart Observer, and it did not occur on Forge on either version.

@Damdrara
Copy link
Author

Damdrara commented Jul 7, 2023

Without digging too deep, super quick code search finds the related change:

c9114f1

It seems this was specifically changed for the Fabric version due to different fluid mechanics, but it wasn't taken into account that the same label is used for items as well, not just fluids.

@MrGreenTea
Copy link

Yeah I've come across this as well. It seems to be an issue with the translation as it does not differentiate between the Item List and Fluid list:

Here is the key in the Translation file Source

Here it is used for the ValueList: ValueListDisplaySource.java

It is the same as used in FluidList: FluidListDisplaySource.java

@nicolas-rabault
Copy link

I experience the same problem on V0.5.1f

@IThundxr IThundxr removed the 1.19 label Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Issue where something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants