-
-
Notifications
You must be signed in to change notification settings - Fork 76
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
More builtin custom replacements #7
Comments
German umlauts (ä, ö, ü) and the letter ß (“sharp s”) are commonly replaced differently:
So maybe an option for these cases makes sense. If I'm not mistaken the Other characters for special treament could be |
I don't intend to internationalize this module, so the target is English.
👍 |
Just out of curiosity as I'm a little bit confused about slug styles in general^^ |
@CanRau Unicode slugs can be fine. It really depends on your constraints, both company-wise and tech stack wise. I'm happy to add an option to preserve Unicode characters. It's just not a common request. |
Hum, now I'm pretty confused and I think the conversation might be confused, too 😅 So german umlauts are properly replaced already and you don't intent to add support for the swiss variation in I suffered for nothing 😆 Then I believe (it seems) only #37 would be missing for @sindresorhus/slugify to be the perfect slugger for me ❤️ 🤞 |
See my comment in #38 (comment). |
I think a common expectation is to have non-alphanumerical symbols map to an empty string by default. Consider for instance a business name like Daisy’s Diner – I don’t want (I half-expected each array in |
I agree with your specific example, but not that all non-alphanumerical symbols should be empty strings. Open a new issue with this and we can figure out a way to improve the output. |
@sindresorhus What do you think about removing filler words for better SEO? See https://www.reliablesoft.net/slug/ Read https://twitter.com/deadcoder0904/status/1424627186565193734 for TL;DR Or should this be done manually? (which makes more sense tbf) |
It would be nice to have em dash |
Don't have any experience regarding em-dash though it sounds interesting 👏🏼 |
My comment was actually meant for the transliterate repo, but for some reason I mixed it up 🤷 |
Uh yea I see, so seems that issue got kinda forgotten |
Regarding @Haraldson 's comment on |
For popular things that makes sense. Suggestions welcome, but I'm not going to add more than maybe 30. So make it worth it.
Don't do a PR. I'll add them all at once when we've decided on what to include.
The text was updated successfully, but these errors were encountered: