cubing / icons

🖼 Icons for WCA events, unofficial events, and competition-related concepts.
https://icons.cubing.net
MIT License
43 stars 22 forks source link

Add 3x3x3 Team-Blind icon #106

Closed dmint789 closed 1 year ago

dmint789 commented 1 year ago

EDIT: The version of the icon with an X was left out of this PR in the end

This solves issue #100. The design was briefly discussed in issue #105. The convention will now be that one circle represents one person, which other team event icons can also follow. Team-Blind Old Style is intended to be used here. Old Style refers to Team-Blind with inspection, whereas the new format on cubingcontests.com is without it, so I want to use this old style icon for the old format.

Previews:

333tbf

333tbfo

lgarron commented 1 year ago

I am unfortunately opposed to an "old" version — this event is unofficial, so there isn't really an "old" version of the event. I also intend to keep advocating for "across the table" Team BLD to make it more practical to judge, and wouldn't want to see more and more "old" versions of events that aren't even official yet.

dmint789 commented 1 year ago

I am unfortunately opposed to an "old" version — this event is unofficial, so there isn't really an "old" version of the event. I also intend to keep advocating for "across the table" Team BLD to make it more practical to judge, and wouldn't want to see more and more "old" versions of events that aren't even official yet.

I understand, but I just want to use this for my own website. I am not insisting that everyone else uses it.

I do think inspectionless team blind is a better event though, because the way it was with inspection, it just turned into 3bld with one competitor telling the other the full memo, and I think it lost most of its value as a team event because of that. Without inspection it seems like this event will have more variety in terms of methods.

dmint789 commented 1 year ago

I have updated the icons as requested. @jfly is this good to merge now? Preview:

333tbf

jfly commented 1 year ago

I am strongly against an "old" icon for team BLD in this repo at the moment

I agree. I'd prefer to reserve that for events that were once WCA official and are no longer official.

@dmint789, how about we rework this PR to just add one event icon (333_team_bld). It's an unofficial event without regulations, I don't think this repo should be in the business of distinguishing between the various regulations that have been invented for unofficial events.

I would really like to avoid cryptically short event IDs going forward

It's amusing to me that the icons repo might be the place that sets the standard for these codes in the future! I agree that the 3-4 character codes are overly obtuse. I vote for 333_team_bld.

dmint789 commented 1 year ago

We have to at least use kebab-case (333-team-bld) to stay consistent, since 333-mirror-blocks uses kebab-case. @jfly I also REALLY like the idea of cubing icons setting naming standards. You should add a small section about that. I, for one, got confused at first, cause I thought the word "unofficial" was part of the event names. And fine, I'll just use my own icon for old style, you don't need to make the repo confusing due to my website's quirk.

dmint789 commented 1 year ago

Done

jfly commented 1 year ago

You should add a small section about that. I, for one, got confused at first, cause I thought the word "unofficial" was part of the event names

Yeah, I can see how that would be confusing! Would you be up for adding a blurp about the structure of this repo to the README?

dmint789 commented 1 year ago

You should add a small section about that. I, for one, got confused at first, cause I thought the word "unofficial" was part of the event names

Yeah, I can see how that would be confusing! Would you be up for adding a blurp about the structure of this repo to the READMe?

Sure. You mean the main README of the repo? Is that connected to the website's front page?

jfly commented 1 year ago

Sure. You mean the main README of the repo? Is that connected to the website's front page?

After thinking about this a bit, I think this is a bit of a hairier problem than I realized. Let's continue this discussion over on https://github.com/cubing/icons/issues/108.

dmint789 commented 1 year ago

Is this PR good to be merged now? Are we gonna stick with kebab-case or switch to snake_case?

jfly commented 1 year ago

LGTM! Just merged it =)