fix: restore missing caret icon on WorkspaceEnvironmentSelection drop… - #1831
vijyotsilare wants to merge 1 commit into
Conversation
…down
Removed popupIcon={null} which was unconditionally hiding the
Autocomplete's default dropdown caret icon.
Fixes meshery/meshery#21674
Signed-off-by: vijyotsilare <vijyotsilare1515@gmail.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe Workspace environment selector no longer suppresses the ChangesWorkspace environment selector
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~2 minutes Severity of issue fixed: Low Merge Risk: ⚪ Minimal · up to The Assigned Environment selector restores its standard dropdown caret, with no remaining merge-readiness risk identified. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
hiyach28
left a comment
There was a problem hiding this comment.
Thank you for your contribution!
Let’s discuss this during tomorrow’s website call at 5:30 PM IST / 6 AM CT.
If you would, please add it as an agenda item to the meeting minutes :)
There was a problem hiding this comment.
removing popupIcon={null} fixes the root cause, the Assigned Environment selector now shows the same default dropdown arrow as the other selectors. The other components that set popupIcon={null} (InputSearchField, UserSearchFieldInput) are search fields where hiding the arrow is intentional, so nothing similar is left to fix. CI is green.
LGTM 🚀
Description
Fixes the missing dropdown caret icon on the "Assigned Environment" selector in the Workspaces section (
playground.meshery.io/management/workspaces). Every other dropdown in the UI shows a caret on the right side to indicate it's an expandable Autocomplete field; this one was silently missing it.Root Cause
WorkspaceEnvironmentSelection.tsxexplicitly setpopupIcon={null}on the MUIAutocompletecomponent, which unconditionally suppresses the default dropdown arrow icon. There was no conditional logic around it — it simply hid the icon in all cases.Fix
Removed the
popupIcon={null}prop, allowing the Autocomplete's default caret icon to render as expected, consistent with every other dropdown in the app.Testing
meshery/meshery, branchfix/assigned-environment-caret-icon, used only for verification — no code changes there).localhost:3000(Next.js dev build).Screenshots
Before:

After:

Related Issue
Fixes meshery/meshery#21674
Summary by CodeRabbit