JustEnoughItems icon indicating copy to clipboard operation
JustEnoughItems copied to clipboard

add history overlay

Open vfyjxf opened this issue 1 year ago • 3 comments

Same as #3801, except that pr was turned off because I incorrectly updated the downstream

vfyjxf avatar Oct 29 '24 14:10 vfyjxf

It would be better if there is an option to place it on the right side instead of the left side

juzi5201314 avatar Dec 18 '24 09:12 juzi5201314

It would be better if there is an option to place it on the right side instead of the left side

It should be configurable in the future, but in the beginning mezz told me it would be better on the left side, since the browsing history obviously has little to do with the list of browsed material, it's more like a bookmark of a temporary nature.

vfyjxf avatar Dec 21 '24 16:12 vfyjxf

release?

shadessass avatar Jan 03 '25 02:01 shadessass

Thanks for contributing!

I did some polishing to complete this feature:

  • created icons
  • fixed a bug where lookups weren't saved if they came from vanilla guis using R/U hotkeys
  • removed the line animation and made the line darker so it is less distracting
  • made sure the dashed lines have complete dashes and don't get cut off partway
  • added description string for the button
  • ensure the layout updates immediately when the configs change, to avoid overlaps and visual glitches
  • save the history to config file so that it persists between play sessions
  • localized configs

mezz avatar Sep 23 '25 06:09 mezz

Great changes, but I'm a little curious as to why you don't allow lookup history on both sides.

vfyjxf avatar Sep 23 '25 06:09 vfyjxf

I think when implementing things like this, it is important only add options that give a large benefit and are easy to understand.

Having the lookup history on both sides is not very consistent with other GUIs in Minecraft or in JEI, where one thing is always in one place. If we allow the option for both, the GUI is no longer "normal" and may need some additional explanation for people to understand. I do not see a very large benefit for having it on both sides, so I removed the ability in order to reduce complexity.

mezz avatar Sep 23 '25 06:09 mezz