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

fix(NcAppNavigation): Bring back hover state for active element #4960

Merged
merged 1 commit into from Dec 15, 2023

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Dec 14, 2023

☑️ Resolves

Bring back the hover state for active app navigation elements.

🏁 Checklist

  • ⛑️ Tests are included or are not applicable
  • 📘 Component documentation has been extended, updated or is not applicable

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux added bug Something isn't working 3. to review Waiting for reviews feature: app-navigation Related to the app-navigation component regression Regression of a previous working feature labels Dec 14, 2023
@susnux susnux added this to the 8.3.1 milestone Dec 14, 2023
@susnux susnux merged commit 78f5179 into master Dec 15, 2023
15 checks passed
@susnux susnux deleted the fix/app-nav-item-hover branch December 15, 2023 00:42
@ShGKme ShGKme mentioned this pull request Dec 22, 2023
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 bug Something isn't working feature: app-navigation Related to the app-navigation component regression Regression of a previous working feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants