#8428: playwright activating a quick bar mod when you dont have a quick bar shortcut configured

[grahamlangford]

What does this PR do?

• Closes Playwright: Activating a quick bar mod when you don’t have a quick bar shortcut configured #8428

Checklist

• Add jest or playwright tests and/or storybook stories
• Designate a primary reviewer @fungairino

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.41%. Comparing base (6d95047) to head (521e665).
Report is 59 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8429      +/-   ##
==========================================
- Coverage   73.47%   73.41%   -0.06%     
==========================================
  Files        1334     1352      +18     
  Lines       41259    41523     +264     
  Branches     7686     7778      +92     
==========================================
+ Hits        30316    30486     +170     
- Misses      10943    11037      +94     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

Playwright test results - MV2

 42 passed
 2 flaky
 8 skipped

Details

 Open report ↗︎
 52 tests across 18 suites
 11 minutes, 10 seconds
 521e665

Flaky tests

chrome › tests/extensionConsoleActivation.spec.ts › can activate a mod with a database
edge › tests/extensionConsoleActivation.spec.ts › can activate a mod with a database

Skipped tests

chrome › tests/extensionConsoleActivation.spec.ts › can activate a mod with built-in integration
edge › tests/extensionConsoleActivation.spec.ts › can activate a mod with built-in integration
chrome › tests/runtime/sidebarController.spec.ts › sidebar controller › shows focus dialog in top-level frame
edge › tests/runtime/sidebarController.spec.ts › sidebar controller › shows focus dialog in top-level frame
chrome › tests/runtime/sidebarNavigation.spec.ts › sidebar mod panels are persistent during navigation
chrome › tests/runtime/sidebarNavigation.spec.ts › sidebar form and temporary panels are unavailable after navigation
edge › tests/runtime/sidebarNavigation.spec.ts › sidebar mod panels are persistent during navigation
edge › tests/runtime/sidebarNavigation.spec.ts › sidebar form and temporary panels are unavailable after navigation

[github-actions]

Playwright test results - MV3

 49 passed
 3 flaky

Details

 Open report ↗︎
 52 tests across 18 suites
 15 minutes, 55 seconds
 521e665

Flaky tests

chrome › tests/pageEditor/saveMod.spec.ts › can save a standalone trigger mod
chrome › tests/regressions/doNotCloseSidebarOnPageEditorSave.spec.ts › #8104: Do not automatically close the sidebar when saving in the Page Editor
edge › tests/regressions/doNotCloseSidebarOnPageEditorSave.spec.ts › #8104: Do not automatically close the sidebar when saving in the Page Editor

[github-actions]

No loom links were found in the first post. Please add one there if you'd like to it to appear on Slack.

Do not edit this comment manually.

[mnholtz]

Opportunity to refactor runModViaQuickbar in this file, which currently works by trying both modifier keys