scrollable revealjs slides with Numbered lists <10 are slightly cut off due to indentation

Open
#14,411 6 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
52/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Quiet
Tech stack
javascript
Domain
frontend

Research direction

Start with the minimal revealjs example using the global scrollable: true option, then reproduce the numbered-list rendering in Safari and with alternate fonts or the night theme in Chrome and Firefox. Compare the first slide with fewer than 10 items to the slide containing item 10; done means no list numbers are clipped in the affected browsers and font configurations.

Written by the indexing model from the issue text.

Description

bug themes

In Quarto ‘1.9.37’, numbered lists <10, some numbers are still slightly cut off on the left side, with both the global option scrollable: true and for one {.scrollable} slide when viewed in Safari.

Related to #5500, this happens in Safari, but not Firefox or Chrome EDIT for the default theme fonts

EDIT: This same issue happens in Chrome and Firefox with other fonts. Adding to the example below, theme: night or an scss file with just $font-family-sans-serif: 'Nunito Sans'; will produce this issue in Chrome and Firefox.

This also seems related to the fix in #4285 because lists >9 are now indented, and I don't recall seeing this before this issue fix (not sure why this altered the indent for lists <10)

Not sure if this is the same bug or a new one introduced by the fix, but numbered lists with <9 items are now cut off.

Minimal example:

---
format:
    revealjs:
        scrollable: true
---

Renders Incorrect

1. 1
1. 2
1. 3
1. 4
1. 5
1. 6
1. 7
1. 8
1. 9


## 

Renders correctly

1. 1
1. 2
1. 3
1. 4
1. 5
1. 6
1. 7
1. 8
1. 9
1. 10

The second slide renders correctly, but the first has most numbers partially cut off on the left.

Quarto Version 1.5.57

Originally posted by @judgelord in #4283

Dominant language
JavaScript
Stars
6k
Forks
458
Avg merge
1d 9h
Merged PRs (30d)
41

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.

More from quarto-dev/quarto-cli

All issues in quarto-dev/quarto-cli

Similar issues

More JavaScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.