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

Update appnavigationcaption #1863

Merged
merged 1 commit into from
Apr 21, 2021
Merged

Conversation

marcoambrosini
Copy link
Contributor

@marcoambrosini marcoambrosini commented Apr 19, 2021

This pr updates the design of the AppNavigationCaption component and adds the ability to slot actions in it.

Screenshot from 2021-04-19 10-15-16

Signed-off-by: Marco Ambrosini marcoambrosini@pm.me

@marcoambrosini marcoambrosini added 2. developing Work in progress feature: app-navigation Related to the app-navigation component component Component discussion and/or suggestion technical debt labels Apr 19, 2021
@marcoambrosini marcoambrosini self-assigned this Apr 19, 2021
@marcoambrosini marcoambrosini added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Apr 19, 2021
@marcoambrosini
Copy link
Contributor Author

Also I'm realizing that this component is missing copyright and docs section.

@korelstar I can't find a copyright from you in the codebase, is that a deliberate choice?

Copy link
Contributor

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

Looks very nice! :)

@marcoambrosini
Copy link
Contributor Author

Added docs and actions component

@korelstar
Copy link
Contributor

Also I'm realizing that this component is missing copyright and docs section.

@korelstar I can't find a copyright from you in the codebase, is that a deliberate choice?

This is just because I didn't think it reaches the threshold of originality ("Schöpfungshöhe" in German) and because I don't like copyright headers. 😉

Signed-off-by: Marco Ambrosini <marcoambrosini@pm.me>
@marcoambrosini marcoambrosini merged commit 220c657 into master Apr 21, 2021
@marcoambrosini marcoambrosini deleted the update-appnavigationcaption branch April 21, 2021 08:18
@skjnldsv skjnldsv mentioned this pull request Jun 4, 2021
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 component Component discussion and/or suggestion feature: app-navigation Related to the app-navigation component technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants