This is an automated email from the ASF dual-hosted git repository.

riemer pushed a commit to branch fix-layout-buttons-close-data-explorer
in repository https://gitbox.apache.org/repos/asf/streampipes.git

commit 2cbffb52ea9dc16f910f337f08cd954ddf6356dc
Author: Dominik Riemer <[email protected]>
AuthorDate: Mon Oct 27 15:15:07 2025 +0100

    fix: Button margin and close button in data explorer
---
 .../data-explorer-chart-container.component.html   | 27 ++++++++++++----------
 .../save-pipeline/save-pipeline.component.html     |  8 ++-----
 2 files changed, 17 insertions(+), 18 deletions(-)

diff --git 
a/ui/src/app/data-explorer-shared/components/chart-container/data-explorer-chart-container.component.html
 
b/ui/src/app/data-explorer-shared/components/chart-container/data-explorer-chart-container.component.html
index b5e72d37e5..c780b4c03f 100644
--- 
a/ui/src/app/data-explorer-shared/components/chart-container/data-explorer-chart-container.component.html
+++ 
b/ui/src/app/data-explorer-shared/components/chart-container/data-explorer-chart-container.component.html
@@ -134,18 +134,21 @@
                             </button>
                         </sp-time-selector-menu>
                     </mat-menu>
-                    <button
-                        mat-icon-button
-                        (click)="removeWidget()"
-                        [matTooltip]="'Delete Chart' | translate"
-                        *ngIf="editMode && hasDataExplorerWritePrivileges"
-                        [attr.data-cy]="
-                            'remove-' +
-                            configuredWidget.baseAppearanceConfig.widgetTitle
-                        "
-                    >
-                        <mat-icon>clear</mat-icon>
-                    </button>
+                    @if (!dataViewMode) {
+                        <button
+                            mat-icon-button
+                            (click)="removeWidget()"
+                            [matTooltip]="'Delete Chart' | translate"
+                            *ngIf="editMode && hasDataExplorerWritePrivileges"
+                            [attr.data-cy]="
+                                'remove-' +
+                                configuredWidget.baseAppearanceConfig
+                                    .widgetTitle
+                            "
+                        >
+                            <mat-icon>clear</mat-icon>
+                        </button>
+                    }
                 </div>
             }
         </div>
diff --git 
a/ui/src/app/editor/dialog/save-pipeline/save-pipeline.component.html 
b/ui/src/app/editor/dialog/save-pipeline/save-pipeline.component.html
index 684b3a23c6..75ffc1fecb 100644
--- a/ui/src/app/editor/dialog/save-pipeline/save-pipeline.component.html
+++ b/ui/src/app/editor/dialog/save-pipeline/save-pipeline.component.html
@@ -50,10 +50,9 @@
         </div>
     </div>
     <mat-divider></mat-divider>
-    <div class="sp-dialog-actions">
+    <div class="sp-dialog-actions" fxLayoutGap="10px">
         <button
             mat-flat-button
-            color="accent"
             (click)="hide(false)"
             *ngIf="operationCompleted"
         >
@@ -61,7 +60,7 @@
         </button>
         <button
             mat-flat-button
-            color="accent"
+            class="mat-basic"
             data-cy="sp-navigate-to-pipeline-overview"
             (click)="navigateToPipelineOverview()"
             *ngIf="operationCompleted"
@@ -75,11 +74,9 @@
                 operationProgress ||
                 operationCompleted
             "
-            mat-button
             mat-flat-button
             color="accent"
             (click)="savePipeline()"
-            style="margin-right: 10px"
             data-cy="sp-editor-apply"
         >
             Apply
@@ -88,7 +85,6 @@
             *ngIf="
                 !operationProgress && (!operationCompleted || 
!operationSuccess)
             "
-            mat-button
             mat-flat-button
             class="mat-basic"
             (click)="hide(true)"

Reply via email to