-
Notifications
You must be signed in to change notification settings - Fork 834
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
Revert EuiSwitch
conversions
#2255
Revert EuiSwitch
conversions
#2255
Conversation
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 tested locally to make sure everything operated as it did.
@@ -7,6 +7,10 @@ | |||
- Fixed `EuiSuperDatePicker` to update `asyncInterval.isStopped` on a `isPaused` prop change. ([#2250](https://github.com/elastic/eui/pull/2250)) | |||
- Converted table, popover, buttons, pagination, outside click detector, focus trap, context menu, and panel to TypeScript ([#2212](https://github.com/elastic/eui/pull/2212)) | |||
|
|||
**Reverts** |
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.
Should the warning notes be in here for the previous releases or will you just do that when you create the next one?
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.
Next one. I'll add notes once there's a real version that can be pointed to.
You'll be able to see them in master, though, after that release gets published. There'll be a manual CL update to document the out-of-order release
* Revert "Convert `EuiSwitch` to TS (elastic#2243)" This reverts commit 39dbcf6. * Revert "Aligning switch semantics with aria role (elastic#2193)" This reverts commit 76e58de. * CL
Summary
Reverts #2193 and #2243.
Nothing wrong with the changes themselves, but we have more pressing changes to make in Kibana that will be jeopardized by the updates required to
EuiSwitch
consumers.The problem we see in Kibana is specific to Angular form $dirty checking. Moving from
input[type=checkbox]
tobutton[role=switch]
was (correctly) done for a11y, but Angular does not flip dirty state forbutton
elements (or for hacking in aninput[type=hidden]
. The Visualize app relies heavily on this Angular mechanism to enable refresh and save capabilities, and has functional tests that match those expectations. The second TS commit reversion is simply for completeness.This merge commit will be cherry-picked for a release outside the standard master process.
Checklist
- [ ] Checked in dark mode- [ ] Checked in mobile- [ ] Checked in IE11 and Firefox- [ ] Props have proper autodocs- [ ] Added documentation examples- [ ] Added or updated jest tests- [ ] Checked for breaking changes and labeled appropriately- [ ] Checked for accessibility including keyboard-only and screenreader modes