feat(design): Add truncated design to DaffBreadcrumbComponent#4278
feat(design): Add truncated design to DaffBreadcrumbComponent#4278joannalauu wants to merge 7 commits intodevelopfrom
Conversation
|
Blocked by #3998 (needs to be merged before this) |
| <ng-template #fullMenu> | ||
| <daff-menu> | ||
| @for (breadcrumbItem of _fullMenuItems(); track breadcrumbItem; let index = $index) { | ||
| <button daff-menu-item> |
There was a problem hiding this comment.
I have not been able to find a good solution to achieve this without breaking other behaviour.
Right now, the breadcrumb is passed into the menu through
<ng-container [ngTemplateOutlet]="breadcrumbItem.itemRef"></ng-container>
as <a href="/docs">Docs</a>, and there isn't really a way to extract the href and innerHTML while keeping the menu content up-to-date, because the menu is usually not re-rendered when switched to another page.
Perhaps it might be better to just keep the structure as it is, even through it isn't ideal? Please let me know if you have any ideas on an effective solution to this.
1f98924 to
147afb4
Compare
63834e9 to
4b6e3f4
Compare
4b6e3f4 to
5704534
Compare
|
@xelaint I really sorry about not thoroughly review my code before requesting a review. I have fixed the breadcrumb behaviour to work as intended, aside from the daff-menu structure which I have elaborated more in the discussion thread. |


This also deprecates the DaffMenuModule.
PR Checklist
PR Type
Current behavior
Fixes: #3513
Developed on top of #3569
New behavior
Breaking change?
Additional context