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

Uplift this to HTML/CSSOM #2

Closed
annevk opened this issue Aug 27, 2020 · 4 comments
Closed

Uplift this to HTML/CSSOM #2

annevk opened this issue Aug 27, 2020 · 4 comments

Comments

@annevk
Copy link

annevk commented Aug 27, 2020

How do you feel about trying to standardize this? See whatwg/html#2464 for some prior discussion around this functionality.

Also, great work documenting the status quo!

@arai-a
Copy link
Owner

arai-a commented Aug 27, 2020

yeah, it would be nice if we can standardize this, and hopefully make the behavior consistent across browsers,
that will help web-devs.
if we standardize, we should look into the behavior of mobile browsers etc as well tho.

about the IsPopup condition, Firefox's behavior can be modified.
the current behavior is based on the mix of others and previous behavior about when to open new window,
and, for example, changing it to just using width would also work.

@arai-a
Copy link
Owner

arai-a commented Aug 28, 2020

Looks like https://html.spec.whatwg.org/multipage/browsers.html#the-rules-for-choosing-a-browsing-context-given-a-browsing-context-name is the right place to modify,
given that it's talking about popup blocker.

@annevk
Copy link
Author

annevk commented Aug 28, 2020

Yeah, though note that what HTML calls a popup is all things that open in a new tab/window, not just windows with a popup look-and-feel.

@arai-a
Copy link
Owner

arai-a commented Aug 28, 2020

okay, if there's no distinction between window/tab/popup in https://html.spec.whatwg.org/multipage/browsers.html#the-rules-for-choosing-a-browsing-context-given-a-browsing-context-name ,
we can keep using "window open steps" as the modification target,
after https://bugzilla.mozilla.org/show_bug.cgi?id=1661643 gets fixed (that removes "current tab" case).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants