Applied-Energistics-2 icon indicating copy to clipboard operation
Applied-Energistics-2 copied to clipboard

Optionally Use Search Engine from JEI or REI

Open MarkusTieger opened this issue 1 year ago • 5 comments

Allows to use the Search Engine from JEI or REI.

This can be useful because it is annoying when you try to search something with the modid and the name like it is possible in JEI. For example "@ae2 crystal". Without it, it tries to search a mod with the name "ae2 crystal". It is configurable in the Terminal Settings. (opt-out to disable btw)

MarkusTieger avatar Aug 02 '23 13:08 MarkusTieger

Thanks for this PR, this is an interesting feature. I'll need to take an in-depth look at this w.r.t. performance and behavior when JEI/REI are absent.

shartte avatar Aug 02 '23 13:08 shartte

@MarkusTieger Sorry for not coming back to you earlier. I am in the process of moving master to Neoforge, so it'll take a while before I can get to this PR. I also would like to actually refactor the functionality significantly, namely that "use JEI search" (the BIG radio buttons we currently have) should probably use this functionality instead of just hiding our own search box, but then use different filter logic.

All in all: it will take a while 😓

shartte avatar Nov 21 '23 12:11 shartte

@MarkusTieger Sorry for not coming back to you earlier. I am in the process of moving master to Neoforge, so it'll take a while before I can get to this PR. I also would like to actually refactor the functionality significantly, namely that "use JEI search" (the BIG radio buttons we currently have) should probably use this functionality instead of just hiding our own search box, but then use different filter logic.

All in all: it will take a while 😓

no problem

MarkusTieger avatar Nov 21 '23 12:11 MarkusTieger

Any progress? We all very need this change! :)

Relvl avatar May 15 '24 09:05 Relvl

Any progress? We all very need this change! :)

I almost forgot this issue, thanks for reminding me. This week I am busy, but I should be able to update this pr in the next week.

MarkusTieger avatar May 15 '24 11:05 MarkusTieger

Also maybe add JEI sorting function? #7827

Relvl avatar May 23 '24 19:05 Relvl

We no longer support JEI in 1.20.6+ at the moment, so that is not going to work...

shartte avatar May 23 '24 19:05 shartte

I guess this issue is no longer required and https://github.com/AppliedEnergistics/Applied-Energistics-2/pull/7883 will resolve the issue with the bad search?

MarkusTieger avatar Jun 08 '24 17:06 MarkusTieger