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

Add classes to elements that have styling applied to faciliate custom styles #102

Merged
merged 1 commit into from
May 24, 2024

Conversation

kiramclean
Copy link
Member

The current 110% width that gets applied to certain nested values can break the UI for pages rendered with quarto because of its mini TOC on the right-hand side, like this for example:

image

This PR just adds some classes to the relevant divs so that over-riding these styles is easier for users. Another solution might be to also remove the 110% width style. Note that it is also currently possible to fix it using CSS like this:

div:has(> .clay-dataset) {
  width: unset !important;
}

(subsituting .clay-dataset for whatever child class is relevant in the specific notebook one is dealing with). This PR would make it generically possible though and not dependent on the order of blocks in a given notebook.

@daslu
Copy link
Member

daslu commented May 24, 2024

Thanks @kiramclean!

@daslu daslu merged commit 878ea46 into main May 24, 2024
@kiramclean kiramclean deleted the km/add-class-to-nested-lists branch May 27, 2024 17:24
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.

None yet

2 participants