-
-
Notifications
You must be signed in to change notification settings - Fork 566
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(icons): Add gender icons #2607
base: main
Are you sure you want to change the base?
Conversation
Added or changed iconsIcons as codeWorks for: const CircleSmallIcon = createLucideIcon('CircleSmall', [
["circle",{"cx":"12","cy":"12","r":"6"}]
])
const MarsStrokeIcon = createLucideIcon('MarsStroke', [
["path",{"d":"m14 6 4 4"}],
["path",{"d":"M17 3h4v4"}],
["path",{"d":"m21 3-7.75 7.75"}],
["circle",{"cx":"9","cy":"15","r":"6"}]
])
const MarsIcon = createLucideIcon('Mars', [
["path",{"d":"M16 3h5v5"}],
["path",{"d":"m21 3-6.75 6.75"}],
["circle",{"cx":"10","cy":"14","r":"6"}]
])
const NonBinaryIcon = createLucideIcon('NonBinary', [
["path",{"d":"M12 2v10"}],
["path",{"d":"m9 4 6 4"}],
["path",{"d":"m9 8 6-4"}],
["circle",{"cx":"12","cy":"17","r":"5"}]
])
const TransgenderIcon = createLucideIcon('Transgender', [
["path",{"d":"M12 16v6"}],
["path",{"d":"M14 20h-4"}],
["path",{"d":"M18 2h4v4"}],
["path",{"d":"m2 2 7.17 7.17"}],
["path",{"d":"M2 5.355V2h3.357"}],
["path",{"d":"m22 2-7.17 7.17"}],
["path",{"d":"M8 5 5 8"}],
["circle",{"cx":"12","cy":"12","r":"4"}]
])
const VenusAndMarsIcon = createLucideIcon('VenusAndMars', [
["path",{"d":"M10 20h4"}],
["path",{"d":"M12 16v6"}],
["path",{"d":"M17 2h4v4"}],
["path",{"d":"m21 2-5.46 5.46"}],
["circle",{"cx":"12","cy":"11","r":"5"}]
])
const VenusIcon = createLucideIcon('Venus', [
["path",{"d":"M12 15v7"}],
["path",{"d":"M9 19h6"}],
["circle",{"cx":"12","cy":"9","r":"6"}]
]) |
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
Not sure about adding the |
A use case besides the obvious veterinarian healthcare context would be electrical neutral. |
My main justification for originally including it was that it appeared on the Wikipedia article for gender symbols; and, unlike for the Mercury symbol, gave no indication that it was superseded by something different. I think it is used in botany. However, I agree that its use case might be more niche than the others, and I'd be happy to get rid of it, knowing we could always add it in later if there is demand for it. Or it could go into `lab'. |
Pending additional use cases or significant demand
icons/non-binary.json
Outdated
"jamiemlaw" | ||
], | ||
"tags": [ | ||
"gender" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"gender" | |
"bigender", | |
"demigender", | |
"enby", | |
"gender", | |
"genderfluid", | |
"multigender", | |
"nonbinary", | |
"pangender", | |
"plumbus", | |
"polygender", | |
"queer", | |
"sex", | |
"transexual", | |
"transfeminine", | |
"transgender", | |
"transmasculine", | |
"two-spirit", | |
"xenogender" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's getting into the weeds of specific gender identities, which I was hoping to avoid. I can foresee two general problems if we went down that route:
- Such a list of tags will never be complete and always open to debate.
- Some of those terms have their own specific icons; a user searching for, say, "pangender" might conclude that we got the icon incorrect, or that we think the symbols are interchangeable.
I'm happy with adding the un-hyphenated "nonbinary" as well as "enby" though.
closes #560
What is the purpose of this pull request?
Description
Adds a variety of gender icons:
mars
(male)venus
(female)venus-and-mars
(intersex, androgynous, hermaphrodite)mars-stroke
(androgyne, transgender, possibly "other gender")transgender
(transgender, gender inclusivity)nonbinary
circle-small
(genderless)neuter
Care has gone into submitting the fewest number of icons to be gender-inclusive, without being overwhelming. It is important to note that these icons represent genders, not gender identities; the latter are too numerous, and too specific for this icon library. This PR also does not include sexual orientations, which, while similar to gender, are a distinct concept, and therefore out of scope. If there is demand, it can be raised separately.
Other icons that are intentionally not included:
mercury
(historically used for intersex, but since replaced byvenus-and-mars
)mars-stroke-right
andmars-stroke-up
(too similar tomars-stroke
in design, not referenced in the Wikipedia article for gender symbols)Icon use case
mars
,venus
andvenus-and-mars
: the biological sex of a person, animal or plantmars
,venus
andneuter
: grammatical gender of a nounvenus-and-mars
,mars-stroke
,nonbinary
,circle-small
,transgender
,neuter
: ways to express a person's gender beyond male and femaletransgender
: to denote gender inclusivity (all genders)circle-small
: a bullet pointAlternative icon designs
Icon Design Checklist
Concept
Author, credits & license
Naming
icons/[iconName].json
.Design
Before Submitting