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

[stable5] Fix HTML validation of AppNavigationToggle #2918

Merged
merged 2 commits into from
Aug 3, 2022

Conversation

backportbot-nextcloud[bot]
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented Aug 2, 2022

backport of #2846

@backportbot-nextcloud backportbot-nextcloud bot added accessibility Making sure we design for the widest range of people possible, including those who have disabilities bug Something isn't working labels Aug 2, 2022
@backportbot-nextcloud backportbot-nextcloud bot added this to the 5.4.0 milestone Aug 2, 2022
@nickvergessen nickvergessen self-assigned this Aug 2, 2022
Error: Attribute icon not allowed on element button at this point.

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Raimund Schlüßler <raimund.schluessler@mailbox.org>
@nickvergessen nickvergessen added the 3. to review Waiting for reviews label Aug 3, 2022
@nickvergessen
Copy link
Contributor

Fixed the backport and added #2877 from @raimund-schluessler

@nickvergessen nickvergessen merged commit a9db2a9 into stable5 Aug 3, 2022
@nickvergessen nickvergessen deleted the backport/2846/stable5 branch August 3, 2022 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews accessibility Making sure we design for the widest range of people possible, including those who have disabilities bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants