-
Notifications
You must be signed in to change notification settings - Fork 975
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
replace callout w component #6709
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -14,7 +14,7 @@ Running with dbt=1.0.0 | |||
Found 13 models, 2 tests, 1 archives, 0 analyses, 204 macros, 2 operations.... | |||
``` | |||
|
|||
:::info dbt Cloud release tracks | |||
:::note dbt Cloud release tracks | |||
<Snippet path="_config-dbt-version-check" /> |
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.
Hiya @mirnawong1
This is more of a question / suggestion. Just thinking of customers who may be visually impaired (this wasn't part of your PR btw) - is it possible to make the background a different colour as the text in the toggle / expandable is quite hard to read.
Kind Regards
Natalie
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.
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.
Hiya @mirnawong1
Thanks for creating this PR and component. I've approved this for you and left one question but the question isn't related to anything you created in this PR.
Kind Regards
Natalie
replacing hardcoded callouts with the
VersionCallout
component for uniformity and scability.Natalie to add the component to the following pages:
🚀 Deployment available! Here are the direct links to the updated files: