search

cherdt / BingoCardGenerator

A bingo card generator
21 stars 10 forks source link

Remove extraneous commas #15

Open cherdt opened 6 years ago

cherdt commented 6 years ago

Commas are used as word/phrase separators. If someone includes an extraneous comma (",,") or a trailing comma ("item1,item2,item3,") then the board will have blank spaces.

We can search for and remove extra and/or trailing commas.

cherdt commented 6 years ago

This feature already exists! removeEmptyElements should take care of this. How are users managing to create cards with empty spaces?

Possibly they are including a space between commas or after a trailing comma. In which case, some people might want to do this intentionally. I'm not sure this should be fixed.

cherdt commented 5 years ago

Yes this was reported again recently, a user included a trailing comma plus a newline. I think we can safely remove commas that have only whitespace following.