Jupyter cell option cap-location writes caption-<fig-cap-location value> class

Aberta Para iniciantes
  • Algo errado?
#15,281 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
1/5
Tempo estimado
Menos de uma hora
Facilidade para iniciantes
90/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
jupyter, typescript
Domínio
backend

Direção de pesquisa

Em src/core/jupyter/jupyter.ts, inspecione o bloco de classes da célula perto de kCapLoc; a issue aponta que ele lê kFigCapLoc ao criar a classe da legenda. Compare o tratamento da mesma opção em hooks.R. O trabalho estará concluído quando o Jupyter usar o valor cap-location para .caption-* e o exemplo relatado produzir .caption-top .fig-cap-location-margin.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

bug engines-jupyter

Bug description

For a Jupyter cell, the cap-location cell option adds the class caption-<value of fig-cap-location> to the cell div instead of caption-<value of cap-location>.

The code is in src/core/jupyter/jupyter.ts (the cell-class block, near kCapLoc):

if (typeof cell.options[kCapLoc] === "string") {
  classes.push(`caption-${cell.options[kFigCapLoc]}`);   // reads kFigCapLoc, not kCapLoc
}

The knitr hook in hooks.R does it correctly (paste0("caption-", options[["cap-location"]])), so the two engines disagree for the same option.

Reproduction

I ran this on a stored notebook with quarto render t.ipynb --to html --no-execute -M keep-md:true and read the intermediate markdown. One code cell, with a PNG in its stored output, whose source is:

#| cap-location: top
#| fig-cap-location: margin
plot()

Result:

::: {#cell-0 .cell .caption-margin .fig-cap-location-margin execution_count=1}

Expected .caption-top .fig-cap-location-margin.

With cap-location set and fig-cap-location unset, the code above would produce .caption-undefined. I read that from the code and did not run it.

Other implementations that copied this block carry the same bug.

Environment

A development checkout of quarto-cli (quarto --version reports 99.9.9), macOS, Jupyter engine with stored outputs (--no-execute).

Linguagem predominante
JavaScript
Estrelas
6.1k
Forks
478
Merge médio
17h 36min
PRs com merge (30d)
139

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de quarto-dev/quarto-cli

Todas as issues de quarto-dev/quarto-cli

Issues semelhantes

Mais issues de JavaScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.