#258 addresses the memory leak in ItemMobSpawner. Unfortunately the clean up handler is not consistently called on every world unloaded. This PR fixes this issue by moving the call out of that if block
* Added option (default on because it just feels so much better) to instantly close the inventory when it was just opened with it when pressing the inventory close key is the first input the screen receives, letting you flash open and close it again as is common when inspecting inventory
* Forgot to add the vanilla survival inventory to the autofocus allowlist defaults (should this be hardcoded instead?) Added missing loadEnableAutoFocus() call.
* Removed logging entry, added a new autofocus default that tests successful (and it overwrites the vanilla inventory gui so needed anyway for it), shortened config comment.
* Added open to have the NEI search be auto-focused on open. It will automatically blue if the mouse is moved before typing anything so immediate 'r' and such use works as expected.
* Added open to have the NEI search be auto-focused on open. It will automatically blue if the mouse is moved before typing anything so immediate 'r' and such use works as expected.
* Add an allowlist for what classnames to allow the search auto-focus to work on
* sa
* Add `INEIAutoFocusSearchEnable` interface for opting in to autofocus search box by a mod. Moved the autofocus list from config into a standalone file at the loss of live-updating.
* Remove now-superfluous lang entries for search focus prefix allowlist
* Undo intellij automatic star-imports...
---------
Co-authored-by: Martin Robertz <dream-master@gmx.net>
* Store heighthackhandlers.cfg as an internal asset instead of an array of strings
Is copied into the config directory upon first launch. Also, comments
are now allowed (only from the beginning of the file).
* Store serialhandlers.cfg as an internal asset instead of an array of strings
Is copied into the config directory upon first launch. Also, comments
are now allowed (only from the beginning of the file).
Removes GTNH-specific code to be replaced with a configuration file.
* Extend the height hack to also cover guiTop and getRecipePosition
Certain legacy recipe handlers hardcode even more of the NEI recipe
screen layout, requiring only the singular recipe handler switcher
widget at the top, whereas this improved NEI moves the recipe pager to
the top as well expecting recipe handler to move down another 16px. One
such mod that breaks is Immersive Engineering. To fix it we move the top
edge of the gui (guiTop) down by 16px so that the old gap calculation
works correctly. This required special handling in getRecipePosition()
and refreshSlots().
* Fix Immersive Engineering recipe handlers
* Fix Ex Nihilo recipe handlers
* Fix Thermal Expansion recipe handlers
These handlers expect only two recipes to be displayed at a time. This can only be fixed through asm AFAICT.
NEI plugins are initialized in a separate thread. If this takes too long,
the client tick handler might already be running checking the keybindings
in KeyManager while a plugin calls API.addKeyBind() which can lead to a
CME.
Reported in https://github.com/GTNewHorizons/GT-New-Horizons-Modpack/issues/13292
Thanks to @Denostrov for helping me track this down.
* Fix memory leak caused by mob spawner items
* Use a weak hash map instead of manually clearing the map
* Clear manager reference to prevent holding a WorldClient reference through the GUI
* Revert "Use a weak hash map instead of manually clearing the map"
This reverts commit 91bb997250.
* Update build script
* Update build script to import fix for shadow in dev and sources artifacts
* Update fixed build script to shadow csv dependency in dev jar as well
* Moved jitpack
* Move new files
Glease