search

plone / volto

React-based frontend for the Plone Content Management System
https://demo.plone.org/
MIT License
487 stars 664 forks source link

`is-adding-contenttype` bodyclass is dragging the querystring with it #6493

Open sneridagh opened 1 week ago

sneridagh commented 1 week ago 
          @stevepiercy technically speaking it should replace all values of space with -. It's stupid that js has replace and replaceAll as such if you have more than one space only the first is replaced.

Concerning the search results I did a search and I've noticed the spaces https://volto.demo.plone.org/search?SearchableText=News%20Page%20With%20Spaces search-body-classes

Personally, I don't understand why we would set the searchable text as body classes. is-adding-contenttype-?searchabletext=newspage with spaces but maybe @sneridagh can enlighten me :)

Originally posted by @ichim-david in https://github.com/plone/volto/issues/6487#issuecomment-2483673374

stevepiercy commented 1 day ago

@kaushik7kk please read and follow First-time contributors, especially Things not to do, Contributing to Plone, and Contributing to Volto.

Also using your super power of observation, you may see that the green "Open" icon reflects its status.