Skip to content

LG-5697: Remove accessible-forms pack#6175

Merged
aduth merged 1 commit intomainfrom
aduth-lg-5697-accessible-forms
Apr 7, 2022
Merged

LG-5697: Remove accessible-forms pack#6175
aduth merged 1 commit intomainfrom
aduth-lg-5697-accessible-forms

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Apr 7, 2022

Why:

  • Per discovery in LG-5420, it's not necessary.
  • Its behavior is especially conflicting with React-based submission handling, as it globally handles the submit event, so this helps with ongoing FlowStateMachine work.
  • Reduce size and scope of common application bundle

**Why**:

- Per discovery in LG-5420, it's not necessary.
- Its behavior is especially conflicting with React-based submission handling, as it globally handles the submit event, so this helps with ongoing FlowStateMachine work.
- Reduce size and scope of common application bundle

changelog: Internal, Optimization, Remove unnecessary JavaScript
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aduth aduth merged commit 056e97b into main Apr 7, 2022
@aduth aduth deleted the aduth-lg-5697-accessible-forms branch April 7, 2022 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants