Skip to content
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

Investigate why checkboxes don't work in some CI builds. #6003

Closed
na9da opened this issue Nov 24, 2021 · 4 comments
Closed

Investigate why checkboxes don't work in some CI builds. #6003

na9da opened this issue Nov 24, 2021 · 4 comments
Assignees

Comments

@na9da
Copy link
Collaborator

na9da commented Nov 24, 2021

How to reproduce issue:

So far we have found that the generated code for HiddenCheckbox.tsx is different for the working and broken one.

Generated code for working build:

return /*#__PURE__*/_react["default"].createElement(_StyledInput, _extends({
    type: "checkbox"
  }, props, {
    ref: ref,
    "aria-checked": isIndeterminate ? "mixed" : props.checked
  }));

Generate code for broken build:

return /*#__PURE__*/_react["default"].createElement(_StyledInput, {
    type: "checkbox",
    ref: ref,
    "aria-checked": isIndeterminate ? "mixed" : props.checked
  });

See how in the broken build, props don't get passed to createElement. Because props get elided, the hidden input element will not trigger onChange callback passed via props.

We need to narrow down and figure out if this is caused by some updated package or something else.

@wibily wibily mentioned this issue Nov 24, 2021
3 tasks
@wibily
Copy link
Contributor

wibily commented Nov 24, 2021

timebox to 1 day to investigate.
Heissenbug that only affects Mike's machine and CI on handle-external-link

@na9da
Copy link
Collaborator Author

na9da commented Nov 24, 2021

It now affects the build for main too - http://ci.terria.io/main/

@na9da
Copy link
Collaborator Author

na9da commented Nov 24, 2021

na9da added a commit that referenced this issue Nov 25, 2021
This is until there is a final upstream fix for #6003.
@klarstrup
Copy link

just fyi this problem is not in babel-plugin-styled-components 2.0.0 - only 2.0.1

@na9da na9da closed this as completed Nov 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants