Skip to content

Update callouts.qmd #1723

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update callouts.qmd #1723

wants to merge 1 commit into from

Conversation

serialc
Copy link

@serialc serialc commented Jul 24, 2025

Added the 'caution' callout example for completeness and explained the heading behaviour

Added the 'caution' callout example for completeness and explained the heading behaviour
@cderv
Copy link
Collaborator

cderv commented Jul 24, 2025

Callout caution is below :

::: {.callout-caution collapse="true"}
## Expand To Learn About Collapse
This is an example of a 'collapsed' caution callout that can be expanded by the user. You can use `collapse="true"` to collapse it by default or `collapse="false"` to make a collapsible callout that is expanded by default.
:::

It is just collapse by default to show this example
image

@cderv
Copy link
Collaborator

cderv commented Jul 24, 2025

/deploy-preview

Copy link
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1723.quarto.org

🔄 Modified Documents

@cderv
Copy link
Collaborator

cderv commented Jul 24, 2025

We know get one more callout than the number of type
image

Added the 'caution' callout example for completeness

So I am not sure this was missing.

and explained the heading behaviour

However, we could indeed show example of that - the two first are example, but they don't explain it.

So we could review it and adapt, but having two cautions is maybe too much

@serialc
Copy link
Author

serialc commented Jul 24, 2025

Good points. I'll integrate the comments and suggestions and propose something better.
👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants