Make Universal Search more generic and extend it

Denrage commented 2 years ago

This will make the Universal Search Module more generic, so it can support different kinds of searches.

There is now a SearchHandler and a SearchResultItem for every SearchType which handles the differences between the types. The search handler can be selected through a combobox next to the search bar and later on also be selected through writing a prefix before the search term.

Example how it currently looks and works: https://s.denrage.goip.de/3eMCiCYwfww8bjN/direct

~~After the general idea is approved, the to do list will be worked on.~~

Things to do:

[x] ~~Split PvP and PvE and determine which one is meant automatically~~ doenst matter because api isnt splitting it
[x] Select SearchHandler through a prefix (s One Wo - l Divinit)
[x] Add Traits
[x] ~~Add Items (maybe with local caching?)~~ Way too big for this PR. The caching would need changes inside blishhud itself and the search would need to be async at least. Also it's still debateable if a complete ui rework would need to be done
[x] ~~Hero Points, Hearts, Vistas, etc.~~ - Not usefull, bc they either dont have a name or a relevant/working chat code
[x] Custom functions for specific search types (nearest waypoints of landmarks)
[x] Custom Tooltips for different search types (better skill/trait display)
[x] Energy, Initiative, NonUnderwater display for skills (upkeep cost for revenant is not in the API)
[x] ~~Additional Tooltip windows for associated skills for traits~~ Will be done separately
[x] Revise Search algorithm and test for performance
[x] Localization
[x] Update Manifest
[x] Handle option to post directly into the chat

dlamkins commented 2 years ago

This is looking fantastic!

dlamkins commented 2 years ago

This looks absolutely fantastic. Should be just about ready to merge. I want to just have it get some tests to be sure.

In the meantime, do you mind modifying the manifest file to reflect this as version v0.7.0 of the module?

Denrage commented 2 years ago

Totally forgot two things additional to the manifest that still needs to be taken care off

blish-hud / Community-Module-Pack

Make Universal Search more generic and extend it #132