Skip to content
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

remove unnecessary div #917

Closed
wants to merge 1 commit into from
Closed

remove unnecessary div #917

wants to merge 1 commit into from

Conversation

ratnanil
Copy link

This wrapper was added by mistake (?) and is confusing to the reader

This wrapper was added by mistake (?) and is confusing to the reader
@cderv
Copy link
Collaborator

cderv commented Nov 13, 2023

/deploy-preview

@cwickham
Copy link
Collaborator

The div here is intentional - this div appears in the intermediate markdown and prevents the code output from being interpreted as markdown (hence the need for asis).

I do agree that this is not particularly clear from the text. I'm going to suggest we change the sentence before the code block to:

If we had not specified output: asis the output, as seen in the intermediate markdown, would have included Quarto's .cell-output div:

@cwickham
Copy link
Collaborator

Closed in favor of #920

@cwickham cwickham closed this Nov 15, 2023
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.

3 participants