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

[BasicUI] Fix closing h4 tag in sitemap list page #2234

Merged
merged 1 commit into from Dec 26, 2023

Conversation

lolodomo
Copy link
Contributor

Signed-off-by: Laurent Garnier lg.hc@free.fr

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
@lolodomo lolodomo added bug Something isn't working basic ui Basic UI labels Dec 26, 2023
@lolodomo lolodomo requested a review from a team as a code owner December 26, 2023 14:22
@lolodomo
Copy link
Contributor Author

@kaikreuzer : funny that no one saw this before. I guess it could be there since the very initial version of Basic UI ;)

Copy link
Contributor

@florian-h05 florian-h05 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@florian-h05 florian-h05 added this to the 4.2 milestone Dec 26, 2023
@florian-h05 florian-h05 merged commit 09df379 into openhab:main Dec 26, 2023
3 checks passed
@lolodomo lolodomo deleted the basicui_fix_closing_h4 branch December 26, 2023 17:52
@lolodomo
Copy link
Contributor Author

Would be good to also apply this fix to 4.1.x branch.

@florian-h05
Copy link
Contributor

Feel free to cherry pick yourself, I’m away from keyboard.

@kaikreuzer
Copy link
Member

Would be good to also apply this fix to 4.1.x branch.

What bug does it fix (besides clearly being wrong html)?

@lolodomo
Copy link
Contributor Author

What bug does it fix (besides clearly being wrong html)?

None, it looks like browsers are able to handle these errors.

@kaikreuzer
Copy link
Member

Then I don't think we need to backport it to 4.1.x.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
basic ui Basic UI bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants