-
Notifications
You must be signed in to change notification settings - Fork 13.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: Select component refactoring - ColorSchemeControl - Iteration 5 #15555
chore: Select component refactoring - ColorSchemeControl - Iteration 5 #15555
Conversation
…re/select-component-colorschemecontrol-iteration-5
@adam-stasiak @jinghua-qa @junlincc @rosemarie-chiu @rusackas help with testing so much appreciated :) |
Codecov Report
@@ Coverage Diff @@
## master #15555 +/- ##
==========================================
+ Coverage 76.95% 77.12% +0.17%
==========================================
Files 976 976
Lines 51290 51295 +5
Branches 6907 6907
==========================================
+ Hits 39468 39561 +93
+ Misses 11603 11513 -90
- Partials 219 221 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code looks good. Just need to fix the Cypress test.
…onent-colorschemecontrol-iteration-5
/testenv up |
@rusackas Ephemeral environment spinning up at http://34.213.160.177:8080. Credentials are |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM as well, and the ephemeral env is lookin' good!
Ephemeral environment shutdown and build artifacts deleted. |
apache#15555) * Enhance Select * Transition Select to Antd * Update test * Fix Cypress * Change name to aria-label * Update Cypress search val * Test Cypress selection
apache#15555) * Enhance Select * Transition Select to Antd * Update test * Fix Cypress * Change name to aria-label * Update Cypress search val * Test Cypress selection
apache#15555) * Enhance Select * Transition Select to Antd * Update test * Fix Cypress * Change name to aria-label * Update Cypress search val * Test Cypress selection
SUMMARY
It replaces the react-select Select component with the Antdesign one in the ColorSchemeControl component.
BEFORE
ColorSchemeControl-BEFORE.mp4
AFTER
ColorSchemeControl-AFTER.mp4
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION