veelkoov / fuzzrake

The code behind getfursu.it
https://getfursu.it/info
GNU Affero General Public License v3.0
11 stars 3 forks source link

Update icon usage within maker table #244

Open TangerineFloof opened 1 month ago

TangerineFloof commented 1 month ago

Description

This PR updates how we use and display icons within the maker table.

Before:

image

After:

image

The overall goal with this PR was to remove the Legend modal. Within UI and web design, needing a Legend for something is generally an anti-pattern. It means that meaning is located separately from where it's consumed, requiring users to be confused about what something means and cross-reference it in another location. Legends can be removed by improving iconography, annotating information inline, through reinforcement learning, and other factors.

Notable Changes:

Regarding the age icons:

The reason for these changes is simple: icons demand attention. If we put an icon there, your eye will be drawn to it. We also need to consider what goal we're trying to solve for here: how will the user make use of this info, and what question of theirs does it solve? In the case of "unknown age," I don't think we're doing anything for the user there. We're saying "we don't have any data" which isn't actionable, and isn't noteworthy.

In Telegram, we discussed hedging our bets. The idea was that our data was incomplete so "unknown" could represent a minor. I think this could be true, but I don't think this should be enough reason to bring this to the user's attention (at least, not on this page):

I'm writing out my thoughts here because this was a longer conversation in Telegram already and I wanted to present my thoughts altogether.

veelkoov commented 1 month ago

Mix of facts and opinions in radom order:

Let me now checkout this and test.

veelkoov commented 1 month ago

I will try to resolve #103 ASAP and then ask you for merge from develop.

Another issue I have is the "new" icon. I deliberately used the "recently added" label, because it makes a difference. Studios are not required to join getfursu.it. Big studios could not care at all, they are big. It's actually a favor if they join, because they are improving getfursu.it and get nothing from it. To sum up: the icon represents newly added studios, but not new studios. Thus, we need to think if it is OK to label them as "new". Obviously, they are big, so no harm done if for 6 weeks they will receive less attention, but... doesn't this deserve some kind of explanation? Or is there a way of labelling it better? Tiny issue I can ignore if you don't see this as important.

After figuring out the above, I will proceed with code rev. For now: please

./toolbox pcf
./toolbox tcf
veelkoov commented 1 month ago
* I will need to solve [Remove IS_MINOR #103](https://github.com/veelkoov/fuzzrake/issues/103) first so that you don't need to care about the legacy field, and this will also improve the name of the method magically ;)

Done https://github.com/veelkoov/fuzzrake/pull/245. Edit: although I was surprised the field was not referenced already. I must have forgotten about removing its use already.