fix(ui): Update the widget images in persona modal - #31673
Conversation
❌ PR checklist incompleteThis PR cannot be merged until the following are addressed on its linked issue:
The fields live on the linked issue in the Shipping project (open the issue → right sidebar → Projects). After you set them, re-run this check (or push a commit) — issue/project changes do not re-trigger it automatically. Maintainers can bypass this check by adding the |
|
| Count | Rule |
|---|---|
| 4 | sonarjs/cyclomatic-complexity |
| 3 | openmetadata-imports/no-lower-layer-page-imports |
All findings
| Location | Rule | Message | |
|---|---|---|---|
| 🟡 | src/components/MyData/CustomizableComponents/AddWidgetModal/AddWidgetTabContent.tsx:29:1 |
openmetadata-imports/no-lower-layer-page-imports |
Pages are route-level composition modules. Move the shared implementation/type to a lower layer instead of importing a page from here. |
| 🟡 | src/components/MyData/CustomizableComponents/WidgetCard/WidgetCard.tsx:21:1 |
openmetadata-imports/no-lower-layer-page-imports |
Pages are route-level composition modules. Move the shared implementation/type to a lower layer instead of importing a page from here. |
| 🟡 | src/utils/CustomizeDetailPage/CustomizeDetailPageClassBase.ts:57:10 |
sonarjs/cyclomatic-complexity |
{"message":"Function has a complexity of 12 which is greater than 10 authorized.","cost":2,"secondaryLocations":[{"line":57,"column":9,"endLine":57,"endColumn": |
| 🟡 | src/utils/CustomizeDetailPage/CustomizeDetailPageClassBase.ts:91:10 |
sonarjs/cyclomatic-complexity |
{"message":"Function has a complexity of 28 which is greater than 10 authorized.","cost":18,"secondaryLocations":[{"line":91,"column":9,"endLine":91,"endColumn" |
| 🟡 | src/utils/CustomizeMyDataPageClassBase.ts:31:1 |
openmetadata-imports/no-lower-layer-page-imports |
Pages are route-level composition modules. Move the shared implementation/type to a lower layer instead of importing a page from here. |
| 🟡 | src/utils/CustomizeMyDataPageClassBase.ts:110:10 |
sonarjs/cyclomatic-complexity |
{"message":"Function has a complexity of 13 which is greater than 10 authorized.","cost":3,"secondaryLocations":[{"line":110,"column":9,"endLine":110,"endColumn |
| 🟡 | src/utils/CustomizeMyDataPageImageUtils.ts:31:72 |
sonarjs/cyclomatic-complexity |
{"message":"Function has a complexity of 13 which is greater than 10 authorized.","cost":3,"secondaryLocations":[{"line":31,"column":71,"endLine":31,"endColumn" |
Fix locally (fast - only checks files changed in this branch):
make ui-checkstyle-changed
✅ Playwright Results — workflow succeededValidated commit ✅ 594 passed · ❌ 0 failed · 🟡 0 flaky · ⏭️ 0 skipped · 🧰 0 lifecycle flaky PerformanceBlocking targets: ✅ met · Optimization targets: 🟡 in progress Shard-job maxima below are not the full workflow wall time; the linked run includes build, fixture, planning, and reporting. 🕒 Full workflow signal wall (to summary) 50m 17s ⏱️ Max setup 4m 1s · max shard execution 15m 59s · max shard-job elapsed before upload 19m 40s · reporting 5s 🌐 220.93 requests/attempt · 2.89 app boots/UI scenario · 13.87% common-shard skew Optimization targets still in progress:
How to debug locally# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip # view trace |
|
🚦 Removed from the merge queue —
|
Code Review ✅ ApprovedRemoves the size prop from widget image utilities and updates persona modal widget assets. No issues found. OptionsDisplay: compact → Showing less information. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Powered by Gitar — free for open source |
* fix(ui): Update the widget images in persona modal * remove the size prop * lint fix * updated images



Describe your changes:
Screen.Recording.2026-08-17.at.10.22.31.PM.mov
Fixes #
I worked on ... because ...
Type of change:
High-level design:
N/A — small change.
Tests:
Use cases covered
Unit tests
Backend integration tests
Ingestion integration tests
Playwright (UI) tests
Manual testing performed
UI screen recording / screenshots:
Not applicable.
Checklist:
Fixes <issue-number>: <short explanation>Fixes #<issue-number>above.Summary by Gitar
sizeprop from widget image utility functions and components.This will update automatically on new commits.
Greptile Summary
The PR refreshes widget preview images used by the landing-page customization and persona flows.
Confidence Score: 5/5
The PR appears safe to merge.
No blocking failure remains.
Important Files Changed
Reviews (4): Last reviewed commit: "updated images" | Re-trigger Greptile