tabulator icon indicating copy to clipboard operation
tabulator copied to clipboard

Sanitize keywords string

Open dylanFdvhuset opened this issue 1 year ago • 2 comments

Today if we filter using "Keywords", if we have spaces at the beginning, the end or extra spaces in the middle of a string, the search breaks.

I created this to sanitize the string from extra spaces to keep the essential.

dylanFdvhuset avatar Sep 13 '23 12:09 dylanFdvhuset

Hey @dylanFdvhuset

Great idea there, thanks

I will include this in the 5.6 feature release.

Cheers

Oli :)

olifolkerd avatar Feb 04 '24 21:02 olifolkerd

Hey @dylanFdvhuset

Sorry, with the complexity of getting the cell range module integrated in the 5.6 release, i have had to push this to the 5.7 release in march.

Cheers

Oli :)

olifolkerd avatar Feb 18 '24 11:02 olifolkerd

Hey @dylanFdvhuset

Looking into this more, im not sure if this is the right approach, the user can set the separator they want to use, so trimming spaces would be the wrong approach if for example their strings included spaces and they were using a comma seperator.

I think for this to be included it would need to use the separator variable rather than just trim spaces, and would need to be managed through a param rather than be the default behaviour.

Cheers

Oli :)

olifolkerd avatar Mar 17 '24 10:03 olifolkerd