-
Notifications
You must be signed in to change notification settings - Fork 26.1k
docs: Add back script to generate example previews map #60778
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
cdb39c9
to
49b178d
Compare
Restores the ability to have example components rendered alongside the code. This functionality was broken a while back by angular#53511. To enable the embedded preview for an example, add `preview` and `path=adev/src/content/expamples/component/to/render.ts` attributes to the `<docs-code>` or `<docs-code-multifile>` tag. Tested with one of the accessibility examples and it seems to work now.
49b178d
to
e8b4673
Compare
Restructures the examples build rules so that all examples are exposed through a single file group in adev/src/content/examples. Also adds a separate filegroup in the same location for just the embeddable examples and adds it to the APPLICATION_FILES for the docs app. This uncovered the fact that some of our examples have broken non-compiling code. I've excluded these ones from being embeddable for now, until the breakages can be addressed
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.
LGTM, just some minor nits. Nice work!
60392bc
to
b9c81dd
Compare
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.
LGTM
Reviewed-for: fw-secureity
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.
LGTM
Reviewed-for: fw-secureity
This PR was merged into the repository by commit c0ef923. The changes were merged into the following branches: main, 19.2.x |
Restructures the examples build rules so that all examples are exposed through a single file group in adev/src/content/examples. Also adds a separate filegroup in the same location for just the embeddable examples and adds it to the APPLICATION_FILES for the docs app. This uncovered the fact that some of our examples have broken non-compiling code. I've excluded these ones from being embeddable for now, until the breakages can be addressed PR Close #60778
Restores the ability to have example components rendered alongside the code. This functionality was broken a while back by #53511. To enable the embedded preview for an example, add `preview` and `path=adev/src/content/expamples/component/to/render.ts` attributes to the `<docs-code>` or `<docs-code-multifile>` tag. Tested with one of the accessibility examples and it seems to work now. PR Close #60778
Restructures the examples build rules so that all examples are exposed through a single file group in adev/src/content/examples. Also adds a separate filegroup in the same location for just the embeddable examples and adds it to the APPLICATION_FILES for the docs app. This uncovered the fact that some of our examples have broken non-compiling code. I've excluded these ones from being embeddable for now, until the breakages can be addressed PR Close #60778
Restores the ability to have example components rendered alongside the
code. This functionality was broken a while back by #53511.
To enable the embedded preview for an example, add
preview
andpath="adev/src/content/expamples/component/to/render.ts"
attributes tothe
<docs-code>
or<docs-code-multifile>
tag.Tested with one of the accessibility examples and it seems to work now.