Skip to content

Fixes #1298 fix not closed divs and add alt #1299

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

Closed
wants to merge 3 commits into from
Closed

Fixes #1298 fix not closed divs and add alt #1299

wants to merge 3 commits into from

Conversation

dzhibas
Copy link
Contributor

@dzhibas dzhibas commented Oct 2, 2020

Fixes #1298

@dzhibas dzhibas requested a review from a team as a code owner October 2, 2020 13:32
@dzhibas
Copy link
Contributor Author

dzhibas commented Oct 2, 2020

@camelid what do you think of this one? just fixed couple of not closed divs

@camelid
Copy link
Member

camelid commented Oct 2, 2020

It looks good to me, but I am not a maintainer of this project so you will need a review from someone else.

Please squash your changes here as well with git rebase -i $(git merge-base master HEAD) so that it provides a cleaner Git history.

@camelid
Copy link
Member

camelid commented Oct 2, 2020

By the way, can you add "fixes" before the issue number in this PR as well?

@dzhibas dzhibas changed the title 1298 fix not closed divs and add alt Fixes #1298 fix not closed divs and add alt Oct 2, 2020
@AliSajid
Copy link
Contributor

Hi Rust team. This is really old. Should be easy to merge.

@senekor
Copy link
Contributor

senekor commented May 20, 2024

Looks good, but there are some conflicts now. I'll merge if they are resolved.

@dzhibas
Copy link
Contributor Author

dzhibas commented May 21, 2024

@senekor iv lost original branch here is new PR with the fixes with up2date master #1299 this PR can be closed in favor of #1299

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.

Fix DOM not closed html tags
4 participants