Fixes 26694

[satender-kumar-collate]

Describe your changes:

Fixes #26694

I worked on ExploreV1.component.tsx to work up the stale closure in exploreLeftPanel and handled the double navigation in handleAdvanceSearchQuickFiltersChange

Type of change:

• Bug fix
• Improvement
• New feature
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation

High-level design:

N/A — small change.

Tests:

Use cases covered

Unit tests

Playwright (UI) tests

Manual testing performed

UI screen recording / screenshots:

Screen Recording 2026-05-21 at 1.39.41 PM

Checklist:

• I have read the CONTRIBUTING document.
• My PR title is Fixes <issue-number>: <short explanation>
• My PR is linked to a GitHub issue via Fixes #<issue-number> above.
• I have commented on my code, particularly in hard-to-understand areas.
• For JSON Schema changes: I updated the migration scripts or explained why it is not needed.
• For UI changes: I attached a screen recording and/or screenshots above.
• I have added tests (unit / integration / Playwright as applicable) and listed them above.

---

Summary by Gitar

• UI Refactor:
  • Replaced legacy antd buttons and switches in ExploreV1 with ui-core-components and Dropdown for a consistent design language.
  • Updated the explore page layout to include a refined toolbar with Export, Advanced Search, and Deleted toggles grouped under a Tools dropdown.
• Bug fixes:
  • Fixed an export size calculation logic error where scope visibility was not being correctly evaluated.
  • Adjusted ExploreV1.test.tsx to reflect the UI component migrations and updated test interactions accordingly.
• E2E Testing:
  • Added Playwright test cases to verify filter persistence in the explore tree and sort order preservation in the URL.

_{This will update automatically on new commits.}

[github-actions]

Jest test Coverage

UI tests summary

Lines	Statements	Branches	Functions
	63.22% (65736/103966)	43.94% (36036/82010)	46.67% (10585/22678)

[chirag-madlani]

Add playwright test for the covering case

[github-actions]

🟡 Playwright Results — all passed (11 flaky)

✅ 4246 passed · ❌ 0 failed · 🟡 11 flaky · ⏭️ 87 skipped

Shard	Passed	Failed	Flaky	Skipped
✅ Shard 1	299	0	0	4
✅ Shard 2	805	0	0	8
🟡 Shard 3	791	0	5	8
🟡 Shard 4	842	0	3	12
🟡 Shard 5	717	0	2	47
🟡 Shard 6	792	0	1	8

🟡 11 flaky test(s) (passed on retry)

• Features/OntologyExplorerCardinality.spec.ts › edges for cardinality-typed relations appear in the graph edge data (shard 3, 1 retry)
• Features/OntologyExplorerCardinality.spec.ts › stats reflect the cardinality-typed edges in the relation count (shard 3, 1 retry)
• Features/OntologyExplorerE2E.spec.ts › switching back from Hierarchy to Overview restores stats (shard 3, 1 retry)
• Features/OntologyExplorerE2E.spec.ts › toggling edge labels off and back on leaves the graph and cardinality map intact (shard 3, 1 retry)
• Features/RTL.spec.ts › Verify Following widget functionality (shard 3, 1 retry)
• Pages/CustomProperties.spec.ts › Table (shard 4, 1 retry)
• Pages/DataContracts.spec.ts › Create Data Contract and validate for Table (shard 4, 1 retry)
• Pages/DataContractsSemanticRules.spec.ts › Validate Description Rule Is_Set (shard 4, 1 retry)
• Pages/ExplorePageRightPanel_KnowledgeCenter.spec.ts › Should remove user owner for knowledgeCenter (shard 5, 1 retry)
• Pages/ExplorePageRightPanel.spec.ts › Should verify deleted glossary term not visible in selection for topic (shard 5, 1 retry)
• Pages/Lineage/LineageFilters.spec.ts › Verify lineage schema filter selection (shard 6, 1 retry)

📦 Download artifacts

How to debug locally

# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip    # view trace

[gitar-bot]

Code Review ✅ Approved 1 resolved / 1 findings

Refactors ExploreV1 UI components and resolves stale closure issues in exploreLeftPanel. No issues found.

✅ 1 resolved

✅ Bug: exploreLeftPanel useMemo still has incomplete dependencies

📄 openmetadata-ui/src/main/resources/ui/src/components/ExploreV1/ExploreV1.component.tsx:403-408 📄 openmetadata-ui/src/main/resources/ui/src/components/ExploreV1/ExploreV1.component.tsx:415
The PR correctly adds handleQuickFiltersChange to the exploreLeftPanel useMemo dependency array to fix the stale closure for the ExploreTree callback. However, the same useMemo also references activeTabKey, loading, and onChangeSearchIndex inside its body without listing them in the dependency array (line 415). This means the Menu's selectedKeys and onClick handler can also become stale when activeTabKey changes independently of tabItems.

Since this PR is specifically fixing stale closure issues in this memo, it would be worth completing the fix for the other captured variables too.

Options

Display: compact → Showing less information.

Comment with these commands to change:

Compact

gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Gitar}

[sonarqubecloud]

Quality Gate passed for 'open-metadata-ui'

Issues
1333 New issues
0 Accepted issues

Measures
18 Security Hotspots
0.0% Coverage on New Code
3.7% Duplication on New Code

See analysis details on SonarQube Cloud