Skip to content

HtmlUnitNekoDOMBuilder: Better handling of <form></form> enclosures - #538

Merged
rbri merged 4 commits into
HtmlUnit:masterfrom
atnak:form_enclosures
Jan 31, 2023
Merged

HtmlUnitNekoDOMBuilder: Better handling of <form></form> enclosures#538
rbri merged 4 commits into
HtmlUnit:masterfrom
atnak:form_enclosures

Conversation

@atnak

@atnak atnak commented Dec 28, 2022

Copy link
Copy Markdown
Contributor

This PR contains:

  • Fixes to HtmlUnitNekoDOMBuilder for better handling of <form></form> to fix issues with <input> ownership
  • Refactoring HtmlUnitNekoDOMBuilder.addNodeToRightParent() for readability and extensibility

@atnak
atnak force-pushed the form_enclosures branch 6 times, most recently from 35b916f to dc89320 Compare January 18, 2023 18:44
@rbri

rbri commented Jan 30, 2023

Copy link
Copy Markdown
Member

@atnak still a draft or can i merge this?

@atnak

atnak commented Jan 30, 2023

Copy link
Copy Markdown
Contributor Author

Oh wait I just resumed this today. I'm making tests atm

@atnak
atnak marked this pull request as ready for review January 31, 2023 01:59
@atnak

atnak commented Jan 31, 2023

Copy link
Copy Markdown
Contributor Author

@rbri Thanks for the wait, this PR is ready for review / merge now.

@rbri
rbri merged commit 1f97be6 into HtmlUnit:master Jan 31, 2023
@rbri

rbri commented Jan 31, 2023

Copy link
Copy Markdown
Member

Many thanks - as always.
Have merged this and will have a look during the next days.

@rbri

rbri commented Jan 31, 2023

Copy link
Copy Markdown
Member

MalformedHtmlTest now shows some problem - i'm already working on this.

@rbri

rbri commented Feb 1, 2023

Copy link
Copy Markdown
Member

Hope we catch them all - thanks again.

@atnak
atnak deleted the form_enclosures branch February 1, 2023 17:23
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.

2 participants