Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add matches, querySelector, querySelectorAll to DOM, and the KEY_SVG_SELECTOR transcoding hint #86

Merged
merged 2 commits into from
Aug 18, 2023

Conversation

carlosame
Copy link
Member

Implement matches(), querySelector() and querySelectorAll()

Add the KEY_SVG_SELECTOR transcoding hint

If KEY_SVG_SELECTOR is set and the document is HTML, its value as a CSS selector is used to locate the element that will be used as the SVG root.

In the case of the transcoding helper, if the selector argument is null then KEY_SVG_SELECTOR is used, and if also null then the previous behaviour follows.

If you come up with a better name for KEY_SVG_SELECTOR, please comment here.

This new hint is part of the effort to make EchoSVG friendly to embedded SVG images, see #40.

If `KEY_SVG_SELECTOR` is set and the document is HTML, its value as a CSS selector
is used to locate the element that will be used as the SVG root.

In the case of the transcoding helper, if the selector argument is null then
`KEY_SVG_SELECTOR` is used, and if also null then the previous behaviour follows.

This new hint is part of the effort to make EchoSVG friendly to embedded SVG images.
See also: #40
@carlosame carlosame added the enhancement New feature or request label Aug 17, 2023
@carlosame carlosame added this to the 0.3.3 milestone Aug 17, 2023
@carlosame carlosame self-assigned this Aug 17, 2023
@carlosame carlosame merged commit e134351 into master Aug 18, 2023
5 checks passed
@carlosame carlosame deleted the fe-query-selector branch August 18, 2023 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant