-
Notifications
You must be signed in to change notification settings - Fork 639
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
Add slugify function to std #5627
Comments
reference: https://www.npmjs.com/package/slugify |
Sounds like a good addition to |
I can take that one. Looking over the implementation of https://www.npmjs.com/package/slugify, there are many options I think would probably not be necessary: {
replacement: '-', // replace spaces with replacement character, defaults to `-`
remove: undefined, // remove characters that match regex, defaults to `undefined`
lower: false, // convert to lower case, defaults to `false`
strict: false, // strip special characters except replacement, defaults to `false`
locale: 'vi', // language code of the locale to use
trim: true // trim leading and trailing replacement chars, defaults to `true`
}
WDYT? |
I'm in favor of reducing number of options in first pass.
Do you mean we would implement it like all these options are true, or these set to the same as npm:slugify?
I don't think we need to make |
I'd also suggest we would use |
I am not too familiar with |
There seems to be differences between online slugification for example for |
closed by #5646 |
Is your feature request related to a problem? Please describe.
A common part of building web apps includes generating pretty URLs based of a random string like a page title.
Hello World
->hello-world
déjà vu
->deja-vu
Foo & Bar
->foo-and-bar
Describe the solution you'd like
Describe alternatives you've considered
The text was updated successfully, but these errors were encountered: