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

prompt(), confirm(), alert() are slated for deprecation by google #1537

Open
ericscheid opened this issue Aug 9, 2021 · 1 comment
Open
Labels
bug We say this works but it doesn't UI/UX User Interface, user experience

Comments

@ericscheid
Copy link
Collaborator

the Google Chrome team is planning to deprecate alert(), prompt(), and confirm(), the three browser-native ways to surface a modal window to users.

The phrasing of the ticket implies that this is only on cross-domain sites (ie. third-party iframes), which in-and-of itself breaks sites like CodePen. But if you dig deeper into the conversation, you learn that they’re actually planning to do this for all sites and implementations.

https://gomakethings.com/google-vs.-the-web/


In our code base it appears we only use confirm(), and this is for handling deleting brews (our authorship of brews).

Looks like there are a couple of misc. mentions of confirm() and prompt() in some of the other packages we use too.

Tagging as bug-regression, not because we broke it, but because it wasis working and now isn't will be become broken. But not quite just now.

@ericscheid ericscheid added bug-regression UI/UX User Interface, user experience labels Aug 9, 2021
@calculuschild calculuschild added bug We say this works but it doesn't and removed bug-regression labels Dec 22, 2021
@G-Ambatte
Copy link
Collaborator

Looks like this is currently on hold, the Chrome team promised to announce the change several months in advance if/when they decide to move forward with it.

https://groups.google.com/a/chromium.org/g/blink-dev/c/hTOXiBj3D6A/m/JPELPWqUAAAJ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug We say this works but it doesn't UI/UX User Interface, user experience
Projects
None yet
Development

No branches or pull requests

3 participants