quarto-dev / quarto-dev/quarto-cli
`scrolling: true` makes all card fill the container height independently of the content
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 6k
- Forks
- 458
- Avg merge
- 1d 9h
- Merged PRs (30d)
- 41
Description
scrolling: false will lead to grid-template-columns: minmax(3em, 1fr); being set for the cards container.
scrolling: true will lead to grid-template-rows: minmax(3em, 1fr) minmax(3em, 1fr);
Hi here is a reproducible example where the second plot size is not auto-sized, the second plot needs |# fill: false to be dispalyed properly which I feel should be the default with scrolling: true?
---
title: "test"
format:
dashboard:
scrolling: true
---
```{python}
#| title: multiple plots
import matplotlib.pyplot as plt
for i in range(4):
x = [1, 2, 3, 4, 5]
y = [1, 4, 9, 16, 25]
plt.plot(x, y)
plt.show()
```
```{python}
#| title: one plots
import matplotlib.pyplot as plt
x = [1, 2, 3, 4, 5]
y = [1, 4, 9, 16, 25]
plt.plot(x, y)
plt.show()
```
quarto check
Quarto 1.5.29
[✓] Checking versions of quarto binary dependencies...
Pandoc version 3.1.11: OK
Dart Sass version 1.70.0: OK
Deno version 1.41.0: OK
Typst version 0.11.0: OK
[✓] Checking versions of quarto dependencies......OK
[✓] Checking Quarto installation......OK
Version: 1.5.29
Path: /home/alexis/opt/quarto-1.5.29/bin
[✓] Checking tools....................OK
TinyTeX: (not installed)
Chromium: (not installed)
[✓] Checking LaTeX....................OK
Using: Installation From Path
Path: /usr/bin
Version: 2023
[✓] Checking basic markdown render....OK
[✓] Checking Python 3 installation....OK
Version: 3.12.3
Path: /usr/bin/python3
Jupyter: 5.7.2
Kernels: python3, local_pcangsd, ir
[✓] Checking Jupyter engine render....OK
[✓] Checking R installation...........OK
Version: 4.4.0
Path: /usr/lib64/R
LibPaths:
- /home/alexis/R/x86_64-redhat-linux-gnu-library/4.4
- /usr/lib64/R/library
- /usr/share/R/library
knitr: 1.46
rmarkdown: 2.26
[✓] Checking Knitr engine render......OK
Originally posted by @alxsimon in https://github.com/quarto-dev/quarto-cli/discussions/9858#discussioncomment-9667291
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by rendering the provided dashboard example with scrolling: true and compare it with scrolling: false, focusing on the dashboard card grid rules shown in the report. The fix is done when the second plot auto-sizes correctly without requiring |# fill: false, while the scrolling layout still behaves as intended.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- css
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100