Better experience on Bedrock Edition

[Mrcomputer1]

Currently, using Smiley Player Trader from Bedrock Edition via Geyser, some features don't work properly. The plugin is technically possible to use, but it is not ideal and in many cases the player will need to fallback to the /spt sub-commands in order to manage products due to the GUI requiring right-click and similar.

Note

The primary goal of the plugin remains Java Edition, Bedrock Edition is secondary to that (it is a "nice to have" feature). It may not always be possible for Bedrock Edition to have an equal or as good experience to Java Edition due to differences between the two editions.

Known issues from Bedrock Edition

• Some click type (such as right click) do not work. (Issue Classification: Functionality Issue)
• Error chat messages (such as out of stock) while in villager trading GUI can't be seen (Issue Classification: UX Issue)
• Villager trading GUI doesn't update when out of stock (Issue Classification: UX Issue)
• Stained glass panes used in /spt GUI aren't flat on Bedrock and as a result make the GUI look ugly. (Issue Classification: Cosmetic Issue)
• Player heads used in /spt GUI are only Steve, instead of the intended icon, due to being unsupported in Bedrock Edition. (Issue Classification: Cosmetic Issue)

Planned Solutions

• Click Types: For Bedrock Edition players, use a mix of the normal /spt GUIs and Bedrock-specific GUIs using Geyser/Floodgate's Cumulus API (custom GUIs are actually one of the very rare cool features of Bedrock edition, it will be kind of fun to try them out 😛).
• Error Chat Messages in Villager Trading GUI: Use Geyser's Cumulus.
• Villager Trading GUI doesn't update:
  • Investigate why this is happening
  • Resolve the issue, if possible.
• Stained Glass Panes: Either remove the panes or use a different item instead.
• Player Heads: Use Geyser's Cumulus or a different item.

Requirements

• Experience for Bedrock players using Geyser should be better.
• Experience for Java players should remain unchanged, at least unchanged under the default configuration.
• Geyser/Floodgate should be an optional dependency (similar to how Vault is an optional dependency).