Skip to content

Comments

docs(integrations): clarify icons SSR usage in Vue#5894

Open
y-l-g wants to merge 3 commits intonuxt:v4from
y-l-g:v4
Open

docs(integrations): clarify icons SSR usage in Vue#5894
y-l-g wants to merge 3 commits intonuxt:v4from
y-l-g:v4

Conversation

@y-l-g
Copy link
Contributor

@y-l-g y-l-g commented Jan 16, 2026

🔗 Linked issue

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Updates documentation to clarify how to use icons with Vue SSR.

The current documentation incorrectly states that displaying icons with Vue SSR is not supported. This PR corrects this by explaining how to properly use icon components for SSR.

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@y-l-g y-l-g requested a review from benjamincanac as a code owner January 16, 2026 11:17
@github-actions github-actions bot added the v4 #4488 label Jan 16, 2026
@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 16, 2026

npm i https://pkg.pr.new/@nuxt/ui@5894

commit: b226d08

@miguilimzero
Copy link
Contributor

I think it is better to state both things... Nuxt UI documentation suggests string identifier as the first option. Just saying that you must import the component goes against Nuxt UI docs itself.

Having a first paragraph telling that the icons with string identifier is currently not supported, and that if the icon on SSR is really needed, it can be imported as a component one by one, with the code example below.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v4 #4488

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants