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.

Sign up for GitHub

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

Jump to bottom

AppNavigationToggle lacks tooltip #2876

Closed
raimund-schluessler opened this issue Jul 23, 2022 · 1 comment · Fixed by #2877
Closed

AppNavigationToggle lacks tooltip #2876

raimund-schluessler opened this issue Jul 23, 2022 · 1 comment · Fixed by #2877
Labels
bug Something isn't working regression Regression of a previous working feature
Milestone
6.0.0

Comments

@raimund-schluessler
Copy link
Contributor

After #2846 the AppNavigationToggle lacks the tooltip (because it now uses a Button instead of an ActionButton). @nickvergessen
@raimund-schluessler raimund-schluessler added bug Something isn't working regression Regression of a previous working feature labels Jul 23, 2022
@raimund-schluessler raimund-schluessler added this to the 5.3.2 milestone Jul 23, 2022
@nickvergessen
Copy link
Contributor

Right, should just add the aria-label as title or v-tooltip I guess

@raimund-schluessler raimund-schluessler mentioned this issue Jul 23, 2022
Merged
@juliushaertl juliushaertl modified the milestones: 5.3.2, 6.0.0 Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working regression Regression of a previous working feature
Projects
None yet
Milestone
6.0.0
Development

Successfully merging a pull request may close this issue.

Bring back AppNavigationToggle tooltip nextcloud-libraries/nextcloud-vue
3 participants
@nickvergessen @raimund-schluessler @juliushaertl