Overwatch-Item-Tracker icon indicating copy to clipboard operation
Overwatch-Item-Tracker copied to clipboard

Can't filter out a category using the filter

Open Gonzom opened this issue 8 years ago • 8 comments
trafficstars

I was trying to use the filter to filter-out the achievement/special items from the all items list, but found out that no matter what combination i used, they still show up.

Maybe change the logic of how the filter works? At start everything is marked, and you filter out those categories you don't want?

Gonzom avatar Apr 13 '17 15:04 Gonzom

I don't think this would be ideal but if you only wanted to view event items for a hero (or all-class) you'd have to deselect everything. Only option would be to have an exclude filter but i cant think of a nice way to add one.

Js41637 avatar Apr 14 '17 08:04 Js41637

What about having the filter checkbox have three states:

Blank: Current default behavior. Doesn't affect what shows up. Checked/Plus: Hides all items that don't match that filter Crossed/Minus: Hides all items that match that filter

Clicking the checkbox could cycle through the three states in that order

corwinburdick avatar Oct 21 '17 19:10 corwinburdick

Yeah, maybe. Not as easy cause a checkbox only has 2 states, there is no inbetween in Javascript. Probably will require a bit of work to make something like this happen.

Js41637 avatar Oct 21 '17 23:10 Js41637

You can probably kinda sorta hack around that with indeterminate checkboxes

invalidCards avatar Oct 22 '17 00:10 invalidCards

They are visual only

Js41637 avatar Oct 22 '17 00:10 Js41637

Fair enough.

invalidCards avatar Oct 22 '17 00:10 invalidCards

I've thought about this a bit more and think I understand the concept problem here.

Background We have 2 types of items -

  1. Real groups - these are items from an event ("Uprising") or PvP items.
  2. Custom groups - these are groups we created out of the former group - "Pachi", "Achievement", "Hero Icons".

While an item can never be from more than one of group 1, it can belong to group 1 and one or more of group 2.

Use cases Ok, so now how a user might want to view items.

First option would be to say "Show me all the items I'm missing ("Unselected") from "Uprising" and "Summer Games" - this will work fine and currently works.

Second option would be to say "Show me all items I'm missing ("Unselected") from "Uprising" and "Summer Games" which are of "Hero Icons" - this will work fine and currently works.

Third option would be to say "Show me all items I'm missing ("Unselected") from "Uprising" and "Summer Games" which are of "Hero Icons" and "Pachi" - this SHOULD work, but it doesn't as your response from #118 says it uses AND, which can never be.

Suggestion

  1. Divide the filter into 3 sections instead of the current 4: 1a) "Selected"/"Unselected" 1b) Custom groups - "Achievement/Special", "Hero Icons", "Pachi" 1c) Real groups - Event groups, "Competitive" and "Regular" (which should be for all non-event non-pvp items) - place "Regular" first, "Comp" second and the events after.

  2. Change logic for sections b and c to "OR" instead of "AND".

Note: By adding the "Regular" group, you will make filtering out events possible in another way, as now someone can select "Regular" and just add to it the other filters.

Note 2: If this is fixed, issue #118 should be closed also.

Gonzom avatar Nov 08 '17 12:11 Gonzom

Once issue #221 is completed this could now be done with a small tweak to it.

  1. Divide the filter into 3 sections: 1a) "Selected"/"Unselected" 1b) Real groups (by order): "Regular Items", "Special", "Competitive", "Overwatch League", "Summer Games", "Halloween Terror", "Lunar New Year", "Overwatch Archives", "Anniversary" 1c) Custom Groups (by order): "Achievement", "Hero Icons", "Pachi", "Diablo", "Hearthstone", "Heroes of the Storm, "Starcraft", "World of Warcraft"

  2. Change logic for sections b and c to "OR" instead of "AND".

  3. Make selections in group "1b" work like "1a" as in a toggle option - just one could be selected. Entries from Group "1b" are unique in that they are always one or another.

Gonzom avatar May 24 '18 07:05 Gonzom