nrknithin opened a new pull request, #4020:
URL: https://github.com/apache/incubator-kie-tools/pull/4020
`main` has been failing on Linux CI since Sep 9. This PR fixes both causes.
### 1. Stale WebKit screenshots (DMN Editor)
Three snapshots left stale by the palette layout change in #3985,
regenerated.
### 2. Attribute order depends on the browser (`xml-parser-ts`)
Chrome 153 changed `DOMParser` to return `xmlns` declarations sorted
alphabetically in
`Element.attributes`. `xml-parser-ts` builds its JSON from that order and
serializes back in the
same order, so `getContent()` returned the DMN with its namespaces reordered
and the round-trip
test failed.
The parser now takes the attribute order from the XML text rather than from
the DOM, falling back
to the DOM order when the two cannot be matched. It sits in the shared
parser, so all marshallers
and editors pick it up.
### Testing
- `xml-parser-ts` suite green, including new tests for the ordering.
- Verified in the CI container (Ubuntu + Chrome 153): the previously failing
`getContent` tests
now pass.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
