Skip to content

docs(vue): import ionicons after @ionic/vue in the Your First App snippets - #4842

Merged
brandyscarney merged 1 commit into
mainfrom
docs/vue-tutorial-import-order
Nov 12, 2026
Merged

brandyscarney merged 1 commit into
mainfrom
docs/vue-tutorial-import-order

Conversation

@brandyscarney

Copy link
Copy Markdown
Member

What This Does

Updates the Vue "Your First App" snippets to match the tutorial app's import order: ionic-team/tutorial-photo-gallery-vue#70

Changes

  • The Tab2Page.vue snippets in your-first-app.mdx, 2-taking-photos.mdx, and 7-live-reload.mdx import ionicons/icons after @ionic/vue, the order Ionic's lint rules require now that they apply to .vue files in the tutorial app.

Notes

  • The snippet of the starter's generated Tab 2 page is unchanged, so it still matches what ionic start creates.

@vercel

vercel Bot commented Nov 12, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
ionic-docs Ready Ready Preview Nov 12, 2026 2:12pm UTC

Request Review

@thetaPC thetaPC left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
The changes will also need to be done in docs.

@brandyscarney

Copy link
Copy Markdown
Member Author

@thetaPC This is the docs 😬 But the tutorial PR is linked.

@brandyscarney
brandyscarney merged commit f365b3a into main Nov 12, 2026
7 checks passed
@brandyscarney
brandyscarney deleted the docs/vue-tutorial-import-order branch November 12, 2026 16:42

This branch was successfully deployed

1 active deployment
Preview — 18551efc Deployed Nov 12, 2026 by vercel[bot]
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

Successfully merging this pull request may close these issues.

2 participants