Feature request: toggle document

[v-kydela]

If I have a JSON document that is entirely single-quoted and I need to make it double-quoted, I'd have to select every property name and value one at a time. If that's outside the scope of this extension, is there another extension that could toggle quotes for the whole document?

[jeremychone]

Interesting. yes, this would be outside. Now, what you could do, is do a regex multi selection, in the search,

There must be a way to select all matches. And then, toggle it. Should work.

[v-kydela]

That's tricky because Toggle Quotes only seems to work if the select text is exactly between a pair of quotes and not if the selected text includes the quotes.

If I try (["'`]).*?\1 then I get this

That won't work because the match includes the quotes and Toggle Quotes won't accept that

If I try (?<=(["'`])).*?(?=\1) then I get this

That won't work because of the text outside the quotes that still gets recognized as being between quotes (": ")

Finally, I came across this solution: (?<=["])[^"]*(?="(?:(?:[^"]*"){2})*[^"]*$)

It works only for one type of quote at a time, and it's still much more complicated than the other expressions. Would it be possible for the Toggle Quotes extension to be more lenient with what particular text needs to be selected?

[jeremychone]

interesting. I understand the regex challenge.

The issue is that the "ignore selected" text is by design to allow a simple and custom way to "exclude" some text from the toggle change.

Now, perhaps, you can split it two steps. First get names '...':, before the ':, and then, another step with the : ' assuming all is formatted this way.

[v-kydela]

Could Toggle Quotes be split into two commands, "Toggle Quotes: Surrounding" and "Toggle Quotes: Within"? It seems like the "within" option would be even easier to implement than the original "surrounding" option, and it would give us the ability to select all and then toggle quotes for the whole document.

[jeremychone]

This is a good idea. I added at #22.