-
Notifications
You must be signed in to change notification settings - Fork 708
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
Remove dart:html page #6345
Remove dart:html page #6345
Conversation
… to js interop pages
Visit the preview URL for this PR (updated for commit 0253bae): |
@parlough So I removed the |
@srujzs Can you advise on the deprecation wording I left in a couple places:
I'm mostly wondering if I should say " Edit: Another I just added:
|
+1 Good idea to remove it from the sidenav and keep it around for now :) I think it's safe to leave As for the existing redirects in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Marya! Just a few small suggestions to consider.
Continuing our long good bye of dart:html
👋 🥹
Co-authored-by: Parker Lougheed <[email protected]>
I prefer this. While it's true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Marya!
dart:html
page from sidenavdart:js_interop
dart:html
from libraries overview pagedart:js_interop
Fixes #6314