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

Transfer aria-label from nav settings menu to settings menu icon-button #33385

Merged

Conversation

JuliaKirschenheuter
Copy link
Contributor

Signed-off-by: julia.kirschenheuter [email protected]

@JuliaKirschenheuter JuliaKirschenheuter added the 3. to review Waiting for reviews label Jul 27, 2022
Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍 as discussed, thanks

@JuliaKirschenheuter JuliaKirschenheuter force-pushed the fix/transfer-aria-label-from-nav-settings-to-settings-icon branch from f197a6c to f22427b Compare July 28, 2022 09:21
Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍

@artonge
Copy link
Contributor

artonge commented Jul 28, 2022

CI failure seems unrelated

@artonge artonge merged commit ee23f41 into master Jul 28, 2022
@artonge artonge deleted the fix/transfer-aria-label-from-nav-settings-to-settings-icon branch July 28, 2022 12:11
@welcome
Copy link

welcome bot commented Jul 28, 2022

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

@skjnldsv skjnldsv mentioned this pull request Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants