quarto-dev / quarto-dev/quarto-cli

`scrolling: true` makes all card fill the container height independently of the content

Open
#9,907 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

dashboards enhancement themes
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

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.