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

Add icon slot to AppNavigationNew component #2720

Merged
merged 1 commit into from
Jun 3, 2022

Conversation

raimund-schluessler
Copy link
Contributor

This replaces the simple button element in the AppNavigationNew component with the Button component and exposes an icon slot to add a material design icon. I also added a documentation example.

Implements part of #2707.

@raimund-schluessler raimund-schluessler added this to the 6.0.0 milestone Jun 2, 2022
@raimund-schluessler raimund-schluessler added 2. developing Work in progress feature: app-navigation Related to the app-navigation component labels Jun 2, 2022
@raimund-schluessler raimund-schluessler marked this pull request as ready for review June 2, 2022 19:26
@raimund-schluessler raimund-schluessler added 3. to review Waiting for reviews breaking PR that requires a new major version and removed 2. developing Work in progress labels Jun 2, 2022
Signed-off-by: Raimund Schlüßler <raimund.schluessler@mailbox.org>
@marcoambrosini marcoambrosini merged commit 47cc565 into master Jun 3, 2022
@marcoambrosini marcoambrosini deleted the fix/2707/appnavigationnew-icon branch June 3, 2022 09:38
@st3iny st3iny mentioned this pull request Jun 3, 2022
2 tasks
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 breaking PR that requires a new major version feature: app-navigation Related to the app-navigation component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants