This is an automated email from the ASF dual-hosted git repository. riemer pushed a commit to branch fix-duplicate-tree-node-warning in repository https://gitbox.apache.org/repos/asf/streampipes.git
commit f0318f683ecf0dedd8d0cc7b88beb868a10dbf0e Author: Dominik Riemer <[email protected]> AuthorDate: Tue Aug 5 13:56:29 2025 +0200 fix: Avoid duplicate tree node types in asset browser --- .../asset-browser-hierarchy.component.html | 21 +++++++-------------- .../asset-browser-hierarchy.component.scss | 10 ++++++++++ 2 files changed, 17 insertions(+), 14 deletions(-) diff --git a/ui/projects/streampipes/shared-ui/src/lib/components/asset-browser/asset-browser-hierarchy/asset-browser-hierarchy.component.html b/ui/projects/streampipes/shared-ui/src/lib/components/asset-browser/asset-browser-hierarchy/asset-browser-hierarchy.component.html index 7b3735f8c2..ba044f6ad8 100644 --- a/ui/projects/streampipes/shared-ui/src/lib/components/asset-browser/asset-browser-hierarchy/asset-browser-hierarchy.component.html +++ b/ui/projects/streampipes/shared-ui/src/lib/components/asset-browser/asset-browser-hierarchy/asset-browser-hierarchy.component.html @@ -23,25 +23,12 @@ class="sp-tree" #tree > - <mat-tree-node *matTreeNodeDef="let node" matTreeNodeToggle> - <sp-asset-browser-node - class="w-100" - [node]="node" - [assetBrowserData]="assetBrowserData" - [assetSelectionMode]="assetSelectionMode" - [filteredAssetLinkType]="filteredAssetLinkType" - [resourceCount]="resourceCount" - [selectedAsset]="selectedAsset" - (selectedNodeEmitter)="selectNode($event)" - > - </sp-asset-browser-node> - </mat-tree-node> - <mat-nested-tree-node *matTreeNodeDef="let node; when: hasChild"> <div class="mat-tree-node"> <button mat-icon-button matTreeNodeToggle + *ngIf="hasChild(0, node)" [attr.data-cy]="'button-' + node.nodeName" [attr.aria-label]="'Toggle ' + node.nodeName" > @@ -53,6 +40,12 @@ }} </mat-icon> </button> + <span + *ngIf="!hasChild(0, node)" + class="mat-icon-button placeholder-icon" + > + <mat-icon class="invisible">chevron_right</mat-icon> + </span> <sp-asset-browser-node fxFlex="100" [node]="node" diff --git a/ui/projects/streampipes/shared-ui/src/lib/components/asset-browser/asset-browser-hierarchy/asset-browser-hierarchy.component.scss b/ui/projects/streampipes/shared-ui/src/lib/components/asset-browser/asset-browser-hierarchy/asset-browser-hierarchy.component.scss index ef8b945b55..0a17dcbe66 100644 --- a/ui/projects/streampipes/shared-ui/src/lib/components/asset-browser/asset-browser-hierarchy/asset-browser-hierarchy.component.scss +++ b/ui/projects/streampipes/shared-ui/src/lib/components/asset-browser/asset-browser-hierarchy/asset-browser-hierarchy.component.scss @@ -42,3 +42,13 @@ .mat-tree-node:hover { background: var(--color-bg-1); } + +.placeholder-icon { + display: inline-flex; + justify-content: center; + align-items: center; +} + +.placeholder-icon .mat-icon.invisible { + visibility: hidden; +}
