Skip to content

Print nodes without ID in --query-all - #1100

Closed
emersion wants to merge 1 commit into
linebender:mainfrom
emersion:query-all-no-id
Closed

emersion wants to merge 1 commit into
linebender:mainfrom
emersion:query-all-no-id

Conversation

@emersion

Copy link
Copy Markdown
Contributor

This can be useful to figure out the bounding box of an SVG document.

Alternative to #1050

This can be useful to figure out the bounding box of an SVG
document.
@luisbg

luisbg commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Both of these are by @emersion and #1100 is explicitly an alternative to #1050, so I think we should pick one. Both merge clean and pass the suite.

#1050 prints the document bbox directly. #1100 prints every node's bbox and the caller unions them, it never prints the root, so it doesn't directly answer "what's the document bbox". It also changes --query-all output contract, though ID'd lines keep their content and order.

I personally prefer #1050 as additive with no compat risk, but changing the --query-all output is a policy call.

@LaurenzV @nicoburns , preference?

@LaurenzV

LaurenzV commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator

Closing in favor of #1050.

@LaurenzV LaurenzV closed this Oct 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants