-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature Request: Allow many styles #7
Comments
|
no it not used the sames, because there are some for example also in mdx files with an import this also get ignored :/ import VisuallyHidden from "@components/visually-hidden" |
Well, If you want to check MDX files: those are fundamentally different than Markdown, and becoming more so in MDX@2. You may want use |
it is used here: gatsbyjs/gatsby#24372 but the docs are mixed normal markdown and some mdx (but same ending) |
For the original question: I view “Many styles” as technically a new style, and that can be done with a regex. I believe I can close this? For your other question:
When you say “same ending”, I understand that as the same extension ( |
yes - for markdown files and mdx files are used the same file extension is it possible to process all files with |
It is somewhat possible currently, in a couple of weeks with MDX@2 it definitely isn’t. See my earlier reply:
|
Alright, I’ll close this as different styles can be achieved with a custom regex. As you don‘t care for GH teams, I’d suggest: .use(syntaxMentions, {style: /^@\w{1,40}$/}) |
thats not include the |
then add |
thx for the link :) .use(syntaxMentions, {style: /^@[\w-]{1,40}$/}) i was not aware that i can put |
Subject of the feature
we can also allow
style: ['github', 'twitter', customRegEx]
#6 (comment)
Problem
if you have GitHub and Twitter mentions in the docs then it would be nice to have both to check
Expected behavior
use many styles from the settings
Alternatives
call theretext-syntax-mentions
2x times with different optionsEDIT: this is not possible, the last option wins in this case and only the test for
twitter
is run :(The text was updated successfully, but these errors were encountered: