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.

Sign up for GitHub

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

Jump to bottom

Remove JS code associated with the old form #3391

Closed
karlcow opened this issue Jul 14, 2020 · 0 comments · Fixed by #3394 or #3446
Closed

Remove JS code associated with the old form #3391

karlcow opened this issue Jul 14, 2020 · 0 comments · Fixed by #3394 or #3446
Assignees
@ksy36
Labels
lang: JS

Comments

@karlcow
Copy link
Member

karlcow commented Jul 14, 2020

We switched to the wizard form and we need to double check that some of the JS code needs to be removed
@ksy36 ksy36 self-assigned this Jul 15, 2020
miketaylr pushed a commit that referenced this issue Jul 15, 2020
Merge pull request #3394 from webcompat/issue/3391/1
2d0bbb1 
Fixes #3391 - Remove js files for the old form and adjust eslint config
@karlcow karlcow linked a pull request Jul 15, 2020 that will close this issue
Fixes #3391 - Remove js files for the old form and adjust eslint config #3394
Merged
miketaylr pushed a commit that referenced this issue Aug 7, 2020
Issue #3391 - Part 1. Factor out popupHandler into an open and close …
d00858c 
…method
@karlcow karlcow linked a pull request Aug 10, 2020 that will close this issue
Fixes #3391 - Add ability to close modals with Esc key #3446
Merged
miketaylr pushed a commit that referenced this issue Aug 17, 2020
Issue #3391 - Part 1. Factor out popupHandler into an open and close …
034b368 
…method
ksy36 added a commit that referenced this issue Aug 17, 2020
@ksy36
Merge pull request #3446 from /issues/3397/1
f130ccd 
Fixes #3391 - Add ability to close modals with Esc key
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ksy36 ksy36

Labels
lang: JS
Projects
None yet
Milestone
No milestone
2 participants
@karlcow @ksy36