Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

When alert() dialog is dismissed, underlying window does not regain focus #7666

Closed
brad2014 opened this issue Mar 12, 2017 · 3 comments
Closed

Comments

@brad2014
Copy link

  • Did you search for similar issues before submitting this one? Yes

  • Describe the issue you encountered:

If a window generates an alert popup, the window does not regain keyboard focus when the alert is dismissed. Keyboard accelerators for the web page stop working until focus is manually restored (e.g. by clicking on the page with the mouse).

  • Platform (Win7, 8, 10? macOS? Linux distro?): MacOS

  • Brave Version (revision SHA): 1db81cb

  • Steps to reproduce:

    1. Visit http://www.dailykillersudoku.com/main/play/14207
    2. Click to focus on page, Hit "h" (keyboard accelerator for "help") to bring up alert(), dismiss it by clicking on OK.
    3. Hit "h" again.
  • Actual result:
    Nothing happens. Second keypress is ignored.

  • Expected result:
    Hitting "h" should bring up the "help" alert again.

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?

  • Is this an issue in the currently released version? Yes

  • Can this issue be consistently reproduced? Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:

  • Any related issues: Perhaps Viewport doesn't regain focus after using Find #3460

@NejcZdovc
Copy link
Contributor

cc @bsclifton

@bsclifton
Copy link
Member

similar #8742

@luixxiul luixxiul added the bug label Jun 4, 2017
@alexwykoff alexwykoff modified the milestones: 1.1.0, Backlog Nov 1, 2017
@Mohendran
Copy link

@brad2014 @bsclifton
I opened the same website on chrome... It also exhibited the same behaviour.

@NejcZdovc NejcZdovc added the help wanted The PR/issue opener needs help to complete/report the task. label May 7, 2018
@bsclifton bsclifton added stale and removed bug/good-first-bug help wanted The PR/issue opener needs help to complete/report the task. labels Aug 26, 2018
@bsclifton bsclifton removed this from the Triage Backlog milestone Aug 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants