quick-xml icon indicating copy to clipboard operation
quick-xml copied to clipboard

de: Preserve whitespace for all nodes containg a text event

Open jamwil opened this issue 3 months ago • 5 comments

Potential solution (or intermediate solution) to #896.

jamwil avatar Aug 31 '25 14:08 jamwil

No, I think, this is bad decision. We already have poor designed make utility, where spaces handled differently than tabs and that counterintuitive. I think, the originally suggested implementation, which may be enabled by setting, would be better.

Mingun avatar Aug 31 '25 18:08 Mingun

I'm fine with the original suggestion as it certainly will satisfy my requirements, I only propose this route as it accomplishes the same thing but is not destructive in the way you indicated. I do appreciate that broken but predictable is a better position to be in than using opinionated heuristics like this.

jamwil avatar Aug 31 '25 18:08 jamwil

Updated as requested.

jamwil avatar Aug 31 '25 20:08 jamwil

@Mingun I think I did this the way you described. Is there anything missing?

jamwil avatar Sep 11 '25 02:09 jamwil

Yes, the changes are mostly perfect, but I want to have configuration setting to enable them. I also want to make some small changes, which are easier to do myself than describe.

Mingun avatar Sep 11 '25 06:09 Mingun