Fix tooltip padding + merge JEI history from JEI-Utilities by vfyjxf

[tttsaurus]

Fix Padding

JEI History

Note:
My implementation is still compatible with JEI-Utilities. No crash. It will disable itself when JEI-Utilities is installed since it modified the ctor that JEI-Utilities overrides

  public IngredientGrid(GridAlignment alignment) { // Left in for compatibility with JEI Utilities
  	this(new IngredientListBatchRenderer(), alignment, enableHistory:false);
  }

Concerns:

Firstly,

@Override
public <V> void show(IFocus<V> focus) {
	focus = Focus.check(focus);

	if (logic.setFocus(focus)) {
		open();
	}

	IngredientGridHistoryProvider.onSetFocus(focus);
}

I post an event in RecipesGui#show to notify enabled history providers, which looks pretty inelegant. JEI-Utilities does ASM on this.

Secondly, in order to notify all history providers

private static final List<IngredientGridHistoryProvider> GLOBAL_HISTORY_CONTAINER = new ArrayList<>();

public IngredientGridHistoryProvider(boolean enable) {
    this.enable = enable;
    this.guiHistoryIngredientSlots = new IngredientListBatchRenderer();

    GLOBAL_HISTORY_CONTAINER.add(this);
}

I created a static field to record all history providers, which looks rough too, but accessing history providers via a Internal.getRuntime()... chain doesn't look good either.

[WaitingIdly]

is there anything that can be done for other types of ingredients - ie mekanism gas or thaumic jei aspects?

[WaitingIdly]

do we need to have a box? it seems like we could just use a horizontal line above the history section to separate it from the bookmarks (or ingredients)

[WaitingIdly]

should this also have an if (guiHistoryIngredientSlots.isEmpty()) return;? we dont need to display the border for a box that isnt used.

[WaitingIdly]

how reasonable would having this as a config be? perhaps it could be edited in-game via dragging the border up/downwards.

[WaitingIdly]

currently, if you hover an ingredient in the history it doesnt have a highlighted background.
adding the mouseX and mouseY params + this solves that

IngredientRenderer hoveredHistory = guiHistoryIngredientSlots.getHovered(mouseX, mouseY);
if (hoveredHistory != null) {
    hoveredHistory.drawHighlight();
}

[WaitingIdly]

instead of doing this, perhaps use Internal to set/get bookmark handler? there really should only be one, and i think it would make sense to expose this in that way. would look something like having JeiStarter#load have Internal.setHistory(new IngredientGridHistoryProvider()).