Skip to content

Conversation

@allowing
Copy link
Contributor

@allowing allowing commented Apr 13, 2022

"ZWNBSP" mean "Zero Width No-Break SPace"

The '\s' does not include it

image

@taylorotwell taylorotwell merged commit 8a10e3f into laravel:9.x Apr 13, 2022
@GrahamCampbell GrahamCampbell changed the title trim zwnbsp [9.x] Trim zwnbsp Apr 13, 2022
@allowing allowing deleted the trim_zwnbsp branch April 14, 2022 05:44
@rodrigopedra
Copy link
Contributor

Hi @allowing would you send the same change to the recently added Str::squish()?

I basically ported the same fix I used here before on PR #41877

I guess it would be nice to keep both places consistent. Unfortunately this week I won't have time to work on this.

@allowing
Copy link
Contributor Author

@rodrigopedra :)
I may forget to reply to you, which has been implemented here. Supplement the reply for the convenience of other netizens. #41971

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants