Skip to content

Add names for open-headed arrows#54

Merged
MDLC01 merged 1 commit intotypst:mainfrom
zacharie-moughanim:main
Mar 30, 2025
Merged

Add names for open-headed arrows#54
MDLC01 merged 1 commit intotypst:mainfrom
zacharie-moughanim:main

Conversation

@zacharie-moughanim
Copy link
Copy Markdown
Contributor

Add names for open-headed arrows

@T0mstone
Copy link
Copy Markdown
Collaborator

How about using .stroked instead of .open?

@T0mstone
Copy link
Copy Markdown
Collaborator

Ah, no, that's already something else, oops. Still, I'd like it for there to be some sort of connection to stroked instead of just introducing an entirely unrelated modifier.

@MDLC01
Copy link
Copy Markdown
Collaborator

MDLC01 commented Mar 12, 2025

Could be .strokedhead (there is already .twohead). But it's a longer name.

@mkorje
Copy link
Copy Markdown
Collaborator

mkorje commented Mar 17, 2025

Yeah that's a bit of a mouthful imo. If we want to keep it related to stroked, how about something like .stroked.head? And can maybe have .open as an alias then?

@MDLC01
Copy link
Copy Markdown
Collaborator

MDLC01 commented Mar 17, 2025

.stroked.head sounds weird to me. In particular, note that .head alone would fallback to that.

@mkorje
Copy link
Copy Markdown
Collaborator

mkorje commented Mar 17, 2025

Ah I forgot about that, not so great then.

@Enivex
Copy link
Copy Markdown
Collaborator

Enivex commented Mar 17, 2025

We already have a billion different meanings for stroked. Open is as good a name as any, and is what the official unicode name uses.

Copy link
Copy Markdown
Collaborator

@MDLC01 MDLC01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's go with .open then

@MDLC01 MDLC01 merged commit 5f13aeb into typst:main Mar 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants