Skip to content

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