-
Notifications
You must be signed in to change notification settings - Fork 32
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
Datahub: Display HTML code for chart web component #500
Conversation
Affected libs:
|
…ermalink and data-view-web-component together displays both components in tabs based on web_component_embedder_url setting for data-view-permalink
GitHub Pages links:
|
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.
Thanks! and sorry for taking so long to review. I thought we could still show the web component export if no embedder url is provided in the config? Not much else, only minor remarks, I've tested it and it works fine, thanks :)
libs/feature/record/src/lib/data-view-share/data-view-share.component.html
Outdated
Show resolved
Hide resolved
libs/feature/record/src/lib/data-view-web-component/data-view-web-component.component.spec.ts
Outdated
Show resolved
Hide resolved
libs/ui/inputs/src/lib/copy-text-button/copy-text-button.component.html
Outdated
Show resolved
Hide resolved
348e7b3
to
2bde180
Compare
Thanks for your feedback @jahow ! I've addressed your comments in the latest commits. |
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.
Looking good, thanks!
PR displays the web component HTML of the currently displayed chart.
To-do:
data-view-permalink
anddata-view-web-component
into one component, including the tabs currently inrecord-metadata
that only displays ifweb_component_embedder_url
setting is set. Or add one intermediate component which only handles the tabs and the display condition (web_component_embedder_url
).