Adds Trinket Compare

[Ellimistdev]

Needed a good way to see a trinket's impact across multiple specs.

Adds:

• Top level nav "Trinkets"
• New Trinket submenu to swap between item, item level, and fight styles

Outstanding todos:

MVP: - Hook into db instead of fetching everything like a goblin - Need to add trinket-specific queries instead of needing to get ALL trinkets for ALL specs 😒 - Server-side data handling

Midterm:

• [x] Better trinket selection dropdown (scrolling at a minimum)
• [x] Item_level handling for non-existent item levels for items
• [x] Update fight_style in chart title.
• [x] Trinket menu colors? Just picked Rogue. is now priest

Longterm:

• Swap item_name for item_id throughout to aid in localization
• Drilldown links: trinket_compare to trinkets

[Bloodmallet]

This looks great! Let me add some information:

• the db does not "know" about simulation results and their inner workings. it is aware of where their jsons exist. therefore fetching all simulation results "like a goblin" is the way to go
• server side tries to be as uninvolved with game data as possible. it tries to let bloodytools in cooperation with simc_support do their thing and just pipe the result through to the frontend js

Midterm:

Better trinket selection dropdown (scrolling at a minimum): [this](https://getbootstrap.com/docs/4.0/components/dropdowns/#options)  should hopefully help
Item_level handling for non-existent item levels for items: when collecting trinket information each trinket should already know about which itemlevels are present for it. this can be used to filter the list or create it
Update fight_style in chart title.: 👍 
Trinket menu colors? Just picked Rogue.: I'd probably pick priest because it's the most neutral "color"

Longterm:

Swap item_name for item_id throughout to aid in localization: 👍 
Drilldown links: trinket_compare to trinkets: sounds great!

[Ellimistdev]

Finally picked this back up!

I reworked this so that it does not constantly self-crawl the site. Given that we currently receive all item data for any one spec, to generate these charts, we only need to hit each spec once and cache the result in localstorage (updated every 30 minutes). Additionally, I extracted several functions from bloodmallet_chart_import() to reduce complexity and improve reusability.

As far as I can tell, these changes don't impact existing functionality, though I have not ventured into accounts and custom charts.

[Bloodmallet]

Sounds great! I'll need some time to review it. I'm currently busy preparing my own chart implementations and that's occupying my attention. I'll hopefully get back to you soon on this. 👍