Unable to dismiss competing modals

[aaemnnosttv]

Bug Description

When a feature tour is triggered at the same time as the insufficient permissions modal, neither one is able to be dismissed using the mouse. The only thing which seems to work is the escape key which then dismisses the feature tour, after which the permissions modal is able to be interacted with as usual.

Steps to reproduce

1. Setup SK
2. With a different admin, connect AdSense
3. With the first admin visit the dashboard while triggering a feature tour

Screenshots

Kapture.2022-12-01.at.22.54.15.mp4

In this scenario, the permissions modal was triggered by the GET:notifications request for AdSense which happens on the dashboard when connected but this issue is not AdSense-specific.

---

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

Acceptance criteria

• It should be possible to dismiss the feature tour OR the permissions modal when one appears on-screen.
  • It would probably be best to hide a feature tour anytime a permissions modal appears on-screen, as the permissions modal is "more important"/"urgent" and having both compete for attention is counter-productive.
  • So ideally, when the permissions modal appears, any active feature tour should be closed (but should be able to appear later, so not permanently dismissed). Focus should be moved to the permissions modal dialog.
  • If that isn't possible, make it so either modal can be interacted with/dismissed.

Implementation Brief

Test Coverage

QA Brief

Changelog entry

[zutigrm]

@aaemnnosttv I can't seem to reproduce this. The repro steps from description seem outdated - since we have shared dashboard, 1 admin can't connect AdSense for another admin (unless my understanding is wrong). But I tried by invoking the modals through console, tried several scenarios, and it works every time. One other difference from the attached video at that time, is also that permission modal prevents scroll now.

Here are the videos of the tests:

Screen.Recording.2023-11-16.at.11.07.23.mov

Screen.Recording.2023-11-16.at.11.08.21.mov

Screen.Recording.2023-11-16.at.11.12.03.mov

[aaemnnosttv]

I'm not sure the issue of competing modals is entirely solved, but it seems like the precondition for this happening was addressed in #5497. We can address this in a new issue if/when it comes up again.