Add the ability to filter ammo chart by my trader levels or price
Reithan opened this issue · comments
Since tarkov tools has my trader levels set in settings, and it has market data from it's API, it would be awesome to be able to set a max price, or max trader level for ammos and be able to see what ammos I should be considering.
Wow, that's a great suggestion! Can't believe I haven't thought of this sooner.
Hello @kokarn can I work on this issue? Unless @Reithan is already working on this one.
Also, here's my idea on implementing this. Since we are having multiple filter categories, we can have a VictoryLegend instance for each category. So for our case we would have 3 legends, "Filter by caliber", "Filter by Trader Level", and "Filter by Price".
If we go ahead with using three VictoryLegend instances, then we could have the legends be more horizontally shaped and ordered to be on top of the graph.
Mock:
The three red boxes represent the legends
Let me know what you think!
Also not really sure what kind of form to use for "Filter by Price", maybe just two input text?
Filter by trader could just be a toggle like it is for other views.
Possible add a toggle to hide ammo that's banned on flea as well.
Filter by price could just be a min-max text entry thing.
I think leaving filter by caliber as-is could be reasonable.
Oh, as to the question if I'm currently working on this, no I am not, @AnyFinCanHappen
@Reithan thanks for the feedback,
Im fine with leaving "filter by caliber" as it is. If we do that I'm not really sure where to put the filter by traders and price range. So for now I just work on the functionality for each of the filters and wait on @kokarn opinion on where we should put the filters.
Also some things to note: I'm not sure if we can have custom components in a VictoryContainer So we may not be able to place the price range and trader filter along side the caliber filter.
Very cool that you want to work on this @AnyFinCanHappen !
I'n general I'm fine with you trying out what you think works best.
If you want my thinking I was thinking more about using the Filter that already exists and putting on top. I've had trouble styling the Legend in the past and I think people want to use all filters combined in the same way they do on other pages.
Wanted to revisit one thing I said:
I said min-max price range, but tbh, I can't think of a super solid reason to want to filter to a min price? Maybe max price by itself would be enough?
I could definitely be wrong though. If you think min would be valuable, it shouldn't be too hard to take or leave it when/if you add max price.
I agree, no need for "min" price which also makes the implementation match the rest of the site 👍
No worries at all! Stuff happens all the time :) Just let me know if you need help and/or support with anything, whenever you get the opportunity :)