-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[docs] display preview warning and decorator info #26819
base: master
Are you sure you want to change the base?
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Deploy preview for dagster-docs ready! Preview available at https://dagster-docs-rictiq58e-elementl.vercel.app Direct link to changed pages: |
@@ -104,6 +104,10 @@ dt > a.reference.internal { | |||
@apply inline-flex items-center px-3 py-0.5 rounded-full align-middle text-xs uppercase font-medium bg-sea-foam text-gable-green | |||
} | |||
|
|||
.preview-tag { |
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.
Do we also need a .preview-tag.flag.preview
for the API docs? Looking at superseded and deprecated, we have two different CSS per flag, but it's not the case for experimental.
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.
That CSS hasn't been touched in quite some time, but it looks like @hellendag was the one to do it. Perhaps he would know. 🤔
2470b5e
to
2b8ada1
Compare
Summary & Motivation
Same as #25362 but for the new preview decorator introduced in #26747