search

sindresorhus / normalize-url

Normalize a URL
MIT License
837 stars 123 forks source link

Add `stripTextFragment` option #130

Closed adam-lynch closed 3 years ago

adam-lynch commented 3 years ago

Resolves #129

sindresorhus commented 3 years ago

Sorry, I think we were talking past each other. It should indeed strip http://sindresorhus.com/about#:~:text=hello => http://sindresorhus.com/about. I thought you were talking about doing this http://sindresorhus.com/about#main:~:text=hello => http://sindresorhus.com/about, which it should not do.

adam-lynch commented 3 years ago

OK I've reverted that bit. I hope it's OK now

sindresorhus commented 3 years ago

Looks good. Thanks 👍