From ed3c79822a0682cdfe26f45aa1ed84848c1bdd3e Mon Sep 17 00:00:00 2001
From: YONGJAE LEE <dev.yongjaelee@gmail.com>
Date: Wed, 31 Dec 2025 23:31:32 +0900
Subject: [PATCH 1/3] fix search menu trigger issue

---
 .../workspace/notebook/action-bar/action-bar.component.html | 3 ++-
 .../workspace/notebook/action-bar/action-bar.component.ts   | 6 ++++++
 .../app/pages/workspace/notebook/notebook.component.html    | 2 +-
 .../workspace/notebook/paragraph/paragraph.component.html   | 1 +
 .../workspace/notebook/paragraph/paragraph.component.ts     | 4 ++--
 5 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/zeppelin-web-angular/src/app/pages/workspace/notebook/action-bar/action-bar.component.html b/zeppelin-web-angular/src/app/pages/workspace/notebook/action-bar/action-bar.component.html
index e2d354995ad..8e3bdea00b8 100644
--- a/zeppelin-web-angular/src/app/pages/workspace/notebook/action-bar/action-bar.component.html
+++ b/zeppelin-web-angular/src/app/pages/workspace/notebook/action-bar/action-bar.component.html
@@ -11,7 +11,7 @@
   -->
 
 <div class="bar" [class.simple]="looknfeel !== 'default'">
-  <div class="title">
+  <div class="title" role="heading" aria-level="1">
     <zeppelin-elastic-input
       nz-tooltip
       [nzTooltipTitle]="note.path"
@@ -182,6 +182,7 @@
       <button
         nz-button
         nz-dropdown
+        [(nzVisible)]="isSearchMenuVisible"
         [nzDropdownMenu]="searchMenu"
         nzTrigger="click"
         nz-tooltip
diff --git a/zeppelin-web-angular/src/app/pages/workspace/notebook/action-bar/action-bar.component.ts b/zeppelin-web-angular/src/app/pages/workspace/notebook/action-bar/action-bar.component.ts
index 4d3cbe86c63..79d2c5af408 100644
--- a/zeppelin-web-angular/src/app/pages/workspace/notebook/action-bar/action-bar.component.ts
+++ b/zeppelin-web-angular/src/app/pages/workspace/notebook/action-bar/action-bar.component.ts
@@ -62,6 +62,7 @@ export class NotebookActionBarComponent extends MessageListenersManager implemen
   editorHide = false;
   commitVisible = false;
   tableHide = false;
+  isSearchMenuVisible = false;
   searchText = '';
   replaceText = '';
   cronOption = [
@@ -251,6 +252,11 @@ export class NotebookActionBarComponent extends MessageListenersManager implemen
     this.handleSearch.emit(searchText);
   }
 
+  openSearchMenu() {
+    this.isSearchMenuVisible = true;
+    this.cdr.markForCheck();
+  }
+
   deleteNote() {
     this.messageService.deleteNote(this.note.id);
   }
diff --git a/zeppelin-web-angular/src/app/pages/workspace/notebook/notebook.component.html b/zeppelin-web-angular/src/app/pages/workspace/notebook/notebook.component.html
index ed0bddd2f2c..c04e5b52e74 100644
--- a/zeppelin-web-angular/src/app/pages/workspace/notebook/notebook.component.html
+++ b/zeppelin-web-angular/src/app/pages/workspace/notebook/notebook.component.html
@@ -90,7 +90,7 @@
             (selected)="onParagraphSelect($event)"
             (triggerSaveParagraph)="saveParagraph($event)"
             (saveNoteTimer)="startSaveTimer()"
-            (searchCode)="actionBar.searchCode()"
+            (openSearchMenu)="actionBar.openSearchMenu()"
           ></zeppelin-notebook-paragraph>
         </div>
       </div>
diff --git a/zeppelin-web-angular/src/app/pages/workspace/notebook/paragraph/paragraph.component.html b/zeppelin-web-angular/src/app/pages/workspace/notebook/paragraph/paragraph.component.html
index 31249c94c4c..10917cc7999 100644
--- a/zeppelin-web-angular/src/app/pages/workspace/notebook/paragraph/paragraph.component.html
+++ b/zeppelin-web-angular/src/app/pages/workspace/notebook/paragraph/paragraph.component.html
@@ -107,6 +107,7 @@
         (sizeChange)="onSizeChange($event)"
         (configChange)="onConfigChange($event, i)"
         [result]="result"
+        [attr.data-testid]="'paragraph-result'"
       ></zeppelin-notebook-paragraph-result>
     </ng-container>
     <zeppelin-notebook-paragraph-footer
diff --git a/zeppelin-web-angular/src/app/pages/workspace/notebook/paragraph/paragraph.component.ts b/zeppelin-web-angular/src/app/pages/workspace/notebook/paragraph/paragraph.component.ts
index 8c9bc262cc2..56a9d4c7f95 100644
--- a/zeppelin-web-angular/src/app/pages/workspace/notebook/paragraph/paragraph.component.ts
+++ b/zeppelin-web-angular/src/app/pages/workspace/notebook/paragraph/paragraph.component.ts
@@ -97,7 +97,7 @@ export class NotebookParagraphComponent
   @Output() readonly triggerSaveParagraph = new EventEmitter<string>();
   @Output() readonly selected = new EventEmitter<string>();
   @Output() readonly selectAtIndex = new EventEmitter<number>();
-  @Output() readonly searchCode = new EventEmitter();
+  @Output() readonly openSearchMenu = new EventEmitter();
 
   private destroy$ = new Subject();
 
@@ -700,7 +700,7 @@ export class NotebookParagraphComponent
   }
 
   handleFindInCode() {
-    this.searchCode.emit();
+    this.openSearchMenu.emit();
   }
 
   ngOnChanges(changes: SimpleChanges): void {

From d0639b0ad7ec7e282c14dc45337560d3cdfa79b6 Mon Sep 17 00:00:00 2001
From: YONGJAE LEE <dev.yongjaelee@gmail.com>
Date: Wed, 31 Dec 2025 23:33:53 +0900
Subject: [PATCH 2/3] prevent folder rename if input is empty by disable button

---
 .../src/app/share/folder-rename/folder-rename.component.html  | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/zeppelin-web-angular/src/app/share/folder-rename/folder-rename.component.html b/zeppelin-web-angular/src/app/share/folder-rename/folder-rename.component.html
index 1de07e99060..f2a9339bb36 100644
--- a/zeppelin-web-angular/src/app/share/folder-rename/folder-rename.component.html
+++ b/zeppelin-web-angular/src/app/share/folder-rename/folder-rename.component.html
@@ -30,7 +30,9 @@
     </nz-form-control>
   </nz-form-item>
   <div class="modal-footer ant-modal-footer">
-    <button nz-button nzType="primary" (click)="rename()">Rename</button>
+    <button nz-button nzType="primary" (click)="rename()" [disabled]="!newFolderPath || !newFolderPath.trim()">
+      Rename
+    </button>
   </div>
 </form>
 

From 34b7b3ba14945e84be8c957601a457a12b44b185 Mon Sep 17 00:00:00 2001
From: YONGJAE LEE <dev.yongjaelee@gmail.com>
Date: Wed, 31 Dec 2025 23:35:07 +0900
Subject: [PATCH 3/3] add accessibility and test attributes

---
 .../notebook/sidebar/sidebar.component.html   |  8 +++-
 .../paragraph/paragraph.component.html        | 38 ++++++++++---------
 .../share/node-list/node-list.component.html  |  4 +-
 3 files changed, 29 insertions(+), 21 deletions(-)

diff --git a/zeppelin-web-angular/src/app/pages/workspace/notebook/sidebar/sidebar.component.html b/zeppelin-web-angular/src/app/pages/workspace/notebook/sidebar/sidebar.component.html
index 9acfe35efab..fde35656973 100644
--- a/zeppelin-web-angular/src/app/pages/workspace/notebook/sidebar/sidebar.component.html
+++ b/zeppelin-web-angular/src/app/pages/workspace/notebook/sidebar/sidebar.component.html
@@ -15,6 +15,7 @@
     <button
       [class.sidebar-button-active]="sidebarState === SidebarState.TOC"
       class="sidebar-button"
+      aria-label="Toggle Table of Contents"
       (click)="setOrToggleSidebarState(SidebarState.TOC)"
     >
       <i nz-icon nzType="unordered-list" nzTheme="outline"></i>
@@ -22,12 +23,17 @@
     <button
       [class.sidebar-button-active]="sidebarState === SidebarState.FILE_TREE"
       class="sidebar-button"
+      aria-label="Toggle File Tree"
       (click)="setOrToggleSidebarState(SidebarState.FILE_TREE)"
     >
       <i nz-icon nzType="folder" nzTheme="outline"></i>
     </button>
   </div>
-  <button class="sidebar-button sidebar-close" (click)="setOrToggleSidebarState(SidebarState.CLOSED)">
+  <button
+    class="sidebar-button sidebar-close"
+    aria-label="Close Sidebar"
+    (click)="setOrToggleSidebarState(SidebarState.CLOSED)"
+  >
     <i *ngIf="sidebarState !== SidebarState.CLOSED" nz-icon nzType="close" nzTheme="outline"></i>
   </button>
   <div class="sidebar-main" *ngIf="sidebarState !== SidebarState.CLOSED">
diff --git a/zeppelin-web-angular/src/app/pages/workspace/published/paragraph/paragraph.component.html b/zeppelin-web-angular/src/app/pages/workspace/published/paragraph/paragraph.component.html
index 7e126e70280..7d1589ba04d 100644
--- a/zeppelin-web-angular/src/app/pages/workspace/published/paragraph/paragraph.component.html
+++ b/zeppelin-web-angular/src/app/pages/workspace/published/paragraph/paragraph.component.html
@@ -9,24 +9,26 @@
   ~ See the License for the specific language governing permissions and
   ~ limitations under the License.
   -->
-<zeppelin-notebook-paragraph-dynamic-forms
-  *ngIf="paragraph"
-  [disable]="paragraph.status === 'RUNNING' || paragraph.status === 'PENDING'"
-  [paramDefs]="paragraph.settings.params"
-  [formDefs]="paragraph.settings.forms"
-  [runOnChange]="paragraph.config.runOnSelectionChange"
-  (formChange)="runParagraph()"
-></zeppelin-notebook-paragraph-dynamic-forms>
-<ng-container *ngIf="paragraph">
-  <zeppelin-notebook-paragraph-result
-    *ngFor="let result of results; index as i; trackBy: trackByIndexFn"
-    [id]="paragraph.id"
-    [published]="true"
-    [currentCol]="paragraph.config.colWidth"
-    [config]="configs[i]"
-    [result]="result"
-  ></zeppelin-notebook-paragraph-result>
-</ng-container>
+<div *ngIf="paragraph" class="paragraph-content">
+  <zeppelin-notebook-paragraph-dynamic-forms
+    [disable]="paragraph.status === 'RUNNING' || paragraph.status === 'PENDING'"
+    [paramDefs]="paragraph.settings.params"
+    [formDefs]="paragraph.settings.forms"
+    [runOnChange]="paragraph.config.runOnSelectionChange"
+    (formChange)="runParagraph()"
+  ></zeppelin-notebook-paragraph-dynamic-forms>
+  <ng-container>
+    <zeppelin-notebook-paragraph-result
+      *ngFor="let result of results; index as i; trackBy: trackByIndexFn"
+      [id]="paragraph.id"
+      [published]="true"
+      [currentCol]="paragraph.config.colWidth"
+      [config]="configs[i]"
+      [result]="result"
+      data-testid="paragraph-result"
+    ></zeppelin-notebook-paragraph-result>
+  </ng-container>
+</div>
 
 <!-- Code Preview Modal Template -->
 <ng-template #codePreviewModal>
diff --git a/zeppelin-web-angular/src/app/share/node-list/node-list.component.html b/zeppelin-web-angular/src/app/share/node-list/node-list.component.html
index 0674090b7b7..f2ab5dc4204 100644
--- a/zeppelin-web-angular/src/app/share/node-list/node-list.component.html
+++ b/zeppelin-web-angular/src/app/share/node-list/node-list.component.html
@@ -36,7 +36,7 @@ <h4>
       (click)="activeNote(node.origin.id)"
     >
       <span class="folder" *ngSwitchCase="'folder'">
-        <a class="name" (click)="toggleFolder(node)">
+        <a class="name" (click)="toggleFolder(node)" [attr.data-testid]="'folder-' + node.title">
           <i nz-icon [nzType]="node.isExpanded ? 'folder-open' : 'folder'" nzTheme="fill"></i>
           {{ node.title }}
         </a>
@@ -122,7 +122,7 @@ <h4>
         </span>
       </span>
       <span class="folder" *ngSwitchCase="'trash'">
-        <a class="name" (click)="toggleFolder(node)">
+        <a class="name" (click)="toggleFolder(node)" [attr.data-testid]="'folder-' + node.title">
           <i nz-icon nzType="delete" nzTheme="fill"></i>
           {{ node.title }}
         </a>