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

Toggle does not function when user clicks outside of the modal #8818

Closed
1 task done
wpdarren opened this issue Jun 5, 2024 · 2 comments
Closed
1 task done

Toggle does not function when user clicks outside of the modal #8818

wpdarren opened this issue Jun 5, 2024 · 2 comments
Labels
P0 High priority Team S Issues for Squad 1 Type: Bug Something isn't working

Comments

@wpdarren
Copy link
Collaborator

wpdarren commented Jun 5, 2024

Bug Description

Reported by @kelvinballoo on the Asana bug bash board.

When the user clicks on the toggle to disable the Enhanced conversion tracking. Rather than clicking on 'Disable' nor 'Cancel' the user clicks outside the modal, the toggle gets stuck and you are not able to toggle on and off.

Video of the issue can be found here

Disabled.toggled.after.no.disabling.mov

Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

  • After clicking on the Enhanced conversion tracking toggle, then clicking outside the modal to close it, it should behave the same as clicking on cancel button - toggle should trigger the modal again after clicking on it

Implementation Brief

  • Update assets/js/components/ModalDialog.js
    • Add onClose prop to the Dialog component and pass it handleDialog callback (already passed as a prop to the main ModalDialog component)

Test Coverage

  • No updates needed

QA Brief

  • Setup Site Kit
  • Connect and setup Ads module. Then go to edit settings of Ads module
  • Enable the enhanced conversion tracking and save it
  • Then click on it to disable it, after modal pops up, click outside of it
  • Once it is closed, click on the toggle again, it should trigger the popup again

Changelog entry

  • Fix the toggle issue that happened when the user closed the popup dialog for enhanced conversion tracking by clicking somewhere outside of the popup.
@wpdarren wpdarren added the Type: Bug Something isn't working label Jun 5, 2024
@binnieshah binnieshah added Team S Issues for Squad 1 P0 High priority Next Up Issues to prioritize for definition labels Jun 6, 2024
@zutigrm zutigrm assigned zutigrm and unassigned zutigrm Jun 6, 2024
@eugene-manuilov eugene-manuilov self-assigned this Jun 6, 2024
@eugene-manuilov
Copy link
Collaborator

IB ✔️

@eugene-manuilov eugene-manuilov removed their assignment Jun 6, 2024
@binnieshah binnieshah removed the Next Up Issues to prioritize for definition label Jun 7, 2024
@zutigrm zutigrm self-assigned this Jun 7, 2024
@zutigrm zutigrm removed their assignment Jun 7, 2024
@benbowler benbowler assigned benbowler and unassigned benbowler Jun 10, 2024
@mohitwp mohitwp self-assigned this Jun 10, 2024
@mohitwp
Copy link
Collaborator

mohitwp commented Jun 11, 2024

QA Update ✅

  • Tested on dev environment.
  • Verified that issue is resolve now and toggle functional now when user clicks outside of the modal.
Recording.1055.mp4

@mohitwp mohitwp removed their assignment Jun 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 High priority Team S Issues for Squad 1 Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

7 participants