-
-
Notifications
You must be signed in to change notification settings - Fork 74
feat(docs,dgeni,daffio): add storefront design docs #4269
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
base: develop
Are you sure you want to change the base?
Conversation
|
Todo:
|
|
@griest024 Is there a way to extract the |
| providers: [ | ||
| provideDaffioDocsDesignSection(DAFF_DOCS_STOREFRONT_PATH), | ||
| ], | ||
| loadChildren: () => import('./design/design.routes').then(r => r.daffioDocsDesignRoutes), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should storefront have it's own routes? i'm having a hard time figuring out where I should be putting the storefront overview page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, I've made them more reusable for storefront and added the overview route
821aff5 to
37a9a9f
Compare
You want a nav link to be an arbitrary component? |
Yes. The |
this can be controlled by an |
will this need to be different in the sidebar? The sidebar nav links are rendered as <a daff-list-item [routerLink]="link.url">{{ link.title }}</a>in the sidebar but <a daffioHeaderItem [routerLink]="link.url" [active]="rla.isActive" routerLinkActive #rla="routerLinkActive">{{ link.title }}</a>in the header. Will these need different components for the design link? |
I think the sidebar may need to turn into a tree where |
okay I've added some basic support for it. the component will the |
|
I think there may have been a misunderstanding on what I'm hoping the UI would be. I'm confused why My original comment about extracting out the link from
My request was (from the original marketing header), how am I able to add This is how the header links are currently generated: I currently don't see a way of pulling the |
The crucial change for you is 5bfacb4#diff-78e1e01dcea1af5d8549dfa9e46bcc5d90b0068d5e2602e8b42de6d91009acc9. The array of nav links now accepts a component type in place of the original plain object. This should allow you to render anything you want. The reason that I created What you need to do is create a component and replace this line:
|
Yeah, can you create a stub component? I'm still confused because Design will no longer be a link. It will be: |
check the value of |
|
I tried (with claude's help) to remove the need for a separate link component and updated the sidebar body to use a tree. 7c37d12 |


PR Checklist
PR Type
Current behavior
Fixes: #4266
Fixes: #4265
New behavior
Breaking change?
Additional context