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

document.getElementsByTagName #64

Merged
merged 22 commits into from
Nov 17, 2023
Merged

document.getElementsByTagName #64

merged 22 commits into from
Nov 17, 2023

Conversation

krichprollsch
Copy link
Member

@krichprollsch krichprollsch commented Oct 25, 2023

We can't simply use the libdom dom_document_get_elements_by_tag_name because it follows an old version of the specifications and returns a NodeList.
Since whatwg/dom@190700b the spec changed in favor of returning an HTMLCollection.

So I'm trying to re-implement the HTMLCollection in zig.

Relates to #18

@krichprollsch krichprollsch self-assigned this Oct 25, 2023
@krichprollsch krichprollsch force-pushed the domdoc-gelementsby branch 2 times, most recently from 0ec6dc5 to 291db12 Compare October 25, 2023 10:06
@krichprollsch krichprollsch changed the title Draft: document.getElementsByTagName document.getElementsByTagName Nov 15, 2023
krichprollsch and others added 12 commits November 15, 2023 16:08
We can't simply use the libdom dom_document_get_elements_by_tag_name
because it follows an old version of the specifications and returns a
NodeList.
Since
whatwg/dom@190700b
the spec changed in favor of returning an HTMLCollection.

So I'm trying to re-implement the HTMLCollection in zig.
The buffer is used for upper case string transformation.
We don't want expose the function to the JS API.
src/netsurf.zig Outdated Show resolved Hide resolved
src/dom/html_collection.zig Outdated Show resolved Hide resolved
src/dom/html_collection.zig Outdated Show resolved Hide resolved
src/dom/html_collection.zig Outdated Show resolved Hide resolved
src/dom/html_collection.zig Outdated Show resolved Hide resolved
src/dom/html_collection.zig Outdated Show resolved Hide resolved
src/dom/html_collection.zig Outdated Show resolved Hide resolved
src/dom/html_collection.zig Outdated Show resolved Hide resolved
src/netsurf.zig Outdated Show resolved Hide resolved
src/netsurf.zig Outdated Show resolved Hide resolved
@francisbouvier francisbouvier merged commit 05d31eb into main Nov 17, 2023
3 checks passed
@francisbouvier francisbouvier deleted the domdoc-gelementsby branch November 17, 2023 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants