Repository navigation
Upgrade Oxygen UI to 0.15.0 and minor console UI changes for unification - #966
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
|
@coderabbitai review |
|
There was a problem hiding this comment.
Copilot review overview
🔵 Needs a closer look
A major UI-library major-version upgrade combined with a cross-cutting form-field rendering change across ~100 files carries visual/interaction regression risk that can only be confirmed through human/visual verification.
Review effort: Balanced
Findings: None
What changed in this PR
This PR upgrades the Oxygen UI component libraries (@wso2/oxygen-ui, -charts-react, -icons-react) from 0.8/0.9.1 to 0.15.0 and the underlying lucide-react from 0.545 to 1.16, re-porting the icon key warning patch and dropping the now-fixed Oxygen patch. Alongside the upgrade, it applies a set of console UI "unification" changes, the largest being a new local components/common/TextField wrapper that renders field labels above the input; ~100 call sites switch their TextField import to this wrapper.
Changes:
- Add
components/common/TextField.tsx(label-above wrapper) and migrate allTextFieldimports to it; convert severalSelectusages toFormControl/FormLabel. - Add
userInitialsutil (+tests) anduseLoadedImagehook; updateAppLayoutavatar initials and footer ordering. - Remove the Data Plane dropdown from Create Environment (auto-select first plane), tidy Create/Edit Environment critical-flag UI, refine
IntegrationScopePickerand the Import Integration re-validate button. - Bump dependency versions and patches in
package.json,pnpm-lock.yaml,pnpm-workspace.yaml.
| File | Description |
|---|---|
components/common/TextField.tsx |
New wrapper rendering label above field; core of the unification change |
hooks/useLoadedImage.ts |
New hook returning a picture URL only once it loads |
utils/string.ts / string.test.ts |
userInitials helper and unit tests |
layouts/AppLayout.tsx |
Avatar initials fallback + footer reorder |
pages/CreateEnvironment.tsx / EditEnvironment.tsx |
Remove data-plane picker; restructure critical-flag UI |
components/IntegrationScopePicker.tsx(.styles.ts) |
Project preselect handling + content-sized layout |
pages/ImportIntegration.tsx |
Re-validate button vertical alignment |
components/** (~90 files) |
Mechanical TextField import swap to common/TextField |
components/MessageBrokers/*, ApiInfo/*, Documents/DocFormPage.tsx |
Select → FormControl+FormLabel conversions |
package.json, pnpm-lock.yaml, pnpm-workspace.yaml, patches/* |
Oxygen/lucide version + patch updates |
Notes from investigation (no blocking inline issues found): removed symbols (SMALL_SELECT_LABEL_SX, InputLabel, IS_CLOUD in EditEnvironment, lowercase Github) have no lingering references; CircularProgress is still used in CreateEnvironment; patch/version changes are internally consistent with no leftover references; the new util follows the repo's utils test convention (hooks/components have no co-located tests). One behavioral point worth human confirmation: Create/Edit Environment now show "Mark as Critical Environment" on cloud, where a now-removed comment previously stated it "does not take effect" — this appears intentional (a matching explanatory Alert was added) but is not called out in the PR description.
Files not reviewed (1)
- ipaas/pnpm-lock.yaml: Generated file
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Purpose
Oxygen UI Upgrade
@wso2/oxygen-ui,@wso2/oxygen-ui-charts-react, and@wso2/oxygen-ui-icons-reactto0.15.0from0.8/0.9.1.lucide-reacticon library from0.545to1.16.Console UI unification Changes
Goals
Approach
User stories
Release note
Documentation
Training
Certification
Marketing
Automation tests
Security checks
Samples
Related PRs
Migrations (if applicable)
Test environment
Learning