Remove the undocumented string form of `link-external-icon`

Aberta Para iniciantes
#15,261 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
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
84/100
Tipo de issue
Refatoração
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
javascript
Domínio
frontend

Direção de pesquisa

Leia o script de links externos em quarto-html-after-body.ejs e localize o ramo que trata os valores de string de link-external-icon. Remova esse comportamento específico para strings, preservando o comportamento booleano; a tarefa estará concluída quando a forma de string não documentada deixar de inserir ícones e as configurações booleanas continuarem funcionando.

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

Descrição

html maintenance
Description

The external-link script in quarto-html-after-body.ejs still has a branch for a string value of link-external-icon.
With a string, it inserts a <i class="bi-<value> quarto-ext-icon"> after each external link instead of using the CSS icon.

This form comes from the original implementation (fc1c503b8) and was never documented.
The schema accepts only true or false, so a string reaches this branch only with validate-yaml: false.
In #9469 the branch got a sibling check, so it no longer adds a second icon when the script runs again after pagination, search or sort.

The proposal is to remove the string branch and keep the boolean behaviour as it is.

  • Related: #9469
Linguagem predominante
JavaScript
Estrelas
6.1k
Forks
478
Merge médio
17h 26min
PRs com merge (30d)
126

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.