-
Notifications
You must be signed in to change notification settings - Fork 3.2k
docs(vscode): update VS Code Extension page to reflect the deprecation of the previous version #4207
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Copilot is correct here: extension id is |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…m/ionic-docs into gm-change-vs-extension
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
![]() |
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.
Looks good to me. FWIW I do think you should have accepted the change from the AI about the capitalization of WebNative because I don't think we should rely on the case insensitive fallback working and it's a very easy change to accept, but it's not worth holding up this fix over.
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.
Pull Request Overview
This PR updates the VS Code Extension documentation to reflect the deprecation of the official Ionic VS Code extension in favor of the community-maintained WebNative extension. The changes ensure consistency with Capacitor's documentation approach and include technical improvements for cross-platform development.
- Updates VS Code extension page to point to WebNative extension instead of deprecated Ionic extension
- Fixes broken links to ionic-webview documentation across multiple versions
- Adds Windows compatibility support for local documentation development
Reviewed Changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
docs/intro/vscode-extension.md | Complete rewrite to reference WebNative extension with install buttons and documentation links |
scripts/release-notes.mjs | Updates file path handling for better Windows compatibility using fileURLToPath and dirname |
src/styles/custom.scss | Adds CSS classes for install and docs buttons used on the VS Code extension page |
docs/core-concepts/webview.md | Fixes broken link to ionic-webview documentation |
versioned_docs/version-v6/core-concepts/webview.md | Fixes broken link to ionic-webview documentation |
versioned_docs/version-v7/core-concepts/webview.md | Fixes broken link to ionic-webview documentation |
versioned_docs/version-v5/native/ionic-webview.md | Adds reference link to Awesome Cordova Plugins documentation |
cspell-wordlist.txt | Adds "webnative" to spell checker dictionary |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
Looks good! 👍
Since the Ionic VS Code extension has been deprecated in favor of the WebNative.dev extension and is no longer maintained, we updated the documentation to point to the right information, aligning with Capacitor's documentation for consistency.
Fix an unresolved link to
ionic-webview
pageAdds support for Windows users to run docs locally
Tested locally - ✅