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

JS Windows should be tab modal, not application modal #1990

Closed
garvankeeley opened this issue May 30, 2016 · 7 comments
Closed

JS Windows should be tab modal, not application modal #1990

garvankeeley opened this issue May 30, 2016 · 7 comments

Comments

@garvankeeley
Copy link
Contributor

Describe the issue you encountered:
Malicious sites will popup a JS modal dialog that blocks input, the browser has to be killed to close the window. You should be able to close the tab.

Expected behavior:
JS popup windows should be Tab-modal, not app-modal. Tab should be closable without interacting with the popup.

Mackeeper popups are an example of this.

@cezaraugusto
Copy link
Contributor

@garvankeeley is anyone already working on it? /cc @diracdeltas

@cezaraugusto cezaraugusto added the needs-info Another team member needs information from the PR/issue opener. label Jun 22, 2016
@diracdeltas
Copy link
Member

cc @bridiver

@bridiver
Copy link
Collaborator

it's doable, be we'd have to rewrite electron's dialog handler to use an NSWindow instead of an NSAlert. afaik there is no way to make a non-window-modal NSAlert

@bridiver
Copy link
Collaborator

actually I take that back. The dialogs are initiated by ipc calls so there is no reason why we can't override and handle ourselves

bridiver added a commit that referenced this issue Jun 22, 2016
@bridiver
Copy link
Collaborator

bridiver commented Jun 22, 2016

should also have a checkbox to prevent future dialog boxes for a site - bravery setting

@luixxiul luixxiul added this to the Backlog milestone Jan 21, 2017
@luixxiul luixxiul removed this from the Backlog milestone May 30, 2017
@luixxiul luixxiul added accessibility and removed needs-info Another team member needs information from the PR/issue opener. labels May 30, 2017
@luixxiul luixxiul reopened this May 30, 2017
@bsclifton
Copy link
Member

@luixxiul this has been closed for some time, right? with #7107

@luixxiul
Copy link
Contributor

I thought this hasn't, but you're right. thanks.

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