Skip to content
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

Ignore colon in words #26

Closed
4 tasks done
fhilgers opened this issue Jul 23, 2024 · 4 comments
Closed
4 tasks done

Ignore colon in words #26

fhilgers opened this issue Jul 23, 2024 · 4 comments
Labels
🙅 no/wontfix This is not (enough of) an issue for this project 👎 phase/no Post cannot or will not be acted on

Comments

@fhilgers
Copy link

Initial checklist

Problem

The german language has different forms of words depending on the gender (example: Mitglieder male, Mitgliederinnen female). One modern variant of writing those, which is required in some formal contexts, is Mitglieder:innen with a colon separating the suffix. Currently this leads to unwanted directives.

Solution

I could not find a test, which explictly tests for a textDirective directly following a word. The only ones I could find was a textDirective directly after an opening bracket, which is essentially the start of a span and not after a letter.

I cannot think of a usecase, where one would want to have a textDirective, directly attached to a word. Also there are already strict requirements for the leafDirectives and containerDirectives to be directly at the start of a line. Also, all directives have to directly follow the colons with no space.

What is the reason for allowing textDirectives directly attached to a word? If there is none, I would like to require that a textDirective is not allowed when attached to a word.

Alternatives

  • A whitelist when handling directives which ignores the unwanted directives.
  • Escaping all uses of colons inside words.
@github-actions github-actions bot added 👋 phase/new Post is being triaged automatically 🤞 phase/open Post is being triaged manually and removed 👋 phase/new Post is being triaged automatically labels Jul 23, 2024
@ChristianMurphy
Copy link
Member

Welcome @fhilgers! 👋
Pretty much all markdown inline constructs can happen mid word


example*example*
example**example**
example`example`

exampleexample
exampleexample
exampleexample


And any of these can be escaped by adding a backslash \.
I don't see a reason why this construct should be different from all the others.

@fhilgers
Copy link
Author

Woops, that makes sense. In my mind the directive syntax felt so much like a function call from general programming languages that I found that behaviour weird. But you're definetly right!

This comment was marked as resolved.

@wooorm wooorm added the 🙅 no/wontfix This is not (enough of) an issue for this project label Jul 30, 2024

This comment was marked as resolved.

@wooorm wooorm added 👎 phase/no Post cannot or will not be acted on and removed 🤞 phase/open Post is being triaged manually labels Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🙅 no/wontfix This is not (enough of) an issue for this project 👎 phase/no Post cannot or will not be acted on
Development

No branches or pull requests

3 participants