Docs: fix StackBlitz examples needing icons #39505
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR takes into account the issue reported in #39499 (the issue has been closed by OP but is still there in our documentation).
As explained in #39499 (comment), the problem is that we don't embed the icons in these cases that come from
site/layouts/partials/icons.html
.The idea of this PR is to add a parameter to handle this specific use case like it's done for the optional JavaScript snippet.
Whenever an
{{ example }}
needs the icons, we only need to use the newstackblitz_add_icons
parameter and set it totrue
like this:{{< example stackblitz_add_icons="true" >}}
.Note: I haven't checked the whole documentation to add this parameter, only the
icon-link
page.Type of changes
Checklist
npm run lint
)Live previews
<svg>
containing the icons shouldn't be in StackBlitz)Related issues
Closes #39499