braziljs / eloquente-javascript

Tradução do livro Eloquent JavaScript - 2ª edição.
5.21k stars 1.19k forks source link

Erro em link no 00-introducao.md e 18-formularios-e-campos-de-formularios.md #432

Closed zignd closed 5 years ago

zignd commented 5 years ago

Após alguns esclarecimentos do @joaostein, estou atualizando a issue então para relatar que vários links do 00-introducao.md estão quebrados e que links do 18-formularios-e-campos-de-formularios.md estão fora do padrão do livro que é a utilização de paths relativos.


Versão anterior da Issue:

No final da linha 158 temos o seguinte:

O [segundo projeto](./chapters/pratica-linguagem-de-programacao.md) conclui a primeira parte do livro.

O link está quebrado, mas usando o original como base, a intenção era apontar para o ./chapters/11-pratica-linguagem-de-programacao.md.

joaostein commented 5 years ago

Obrigado, @zignd!

Dei uma olhada e existem 12 ocorrências similares com links quebrados no capítulo de introdução.

Todos os caminhos que contêm ./chapters/<path> estão quebrando.

(além do caso específico que você pontuou)

Veja a foto:

zignd commented 5 years ago

Olá @joaostein! Fiz uma verificação aqui no meu clone com o seguinte comando:

grep -ri -P "chapters\/\D"

Para trazer apenas os links que após "chapters/" têm algo que não é um digito e retornou duas ocorrências para mim.

Imgur

Posso corrigir essas duas então?

zignd commented 5 years ago

Opa, entendi agora, o problema está em ter "chapters" no path, isso acaba duplicando o termo do path durante a visualização pelo GitHub. Desculpa, não tinha entendido incialmente.

joaostein commented 5 years ago

Sem problemas, @zignd!

zignd commented 5 years ago

Atualizei a issue @joaostein, para contemplar isso e também para incluir o que acabei de identificar no capítulo 18. Que tem links apontando para outros capítulos utilizando uma URL completa. Fiz uma pull request com uma correção. Você poderia validar?

joaostein commented 5 years ago

claro, já está merged! obrigado pela contribuição!