Eliminate horizontal scrolling in rust code blocks
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Bug
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- markdown, rust
- Ambito
- documentation
Direzione di ricerca
Inizia con rustfmt.toml e i blocchi di codice Rust modificabili descritti nell’issue, quindi confronta i blocchi renderizzati con diverse lunghezze e diversi numeri di righe. Esamina come vengono gestiti i file Rust inclusi, i commenti tradotti, le espressioni use, le macro, i comandi shell e l’output. Il completamento dovrebbe includere un approccio concordato che impedisca lo scorrimento orizzontale e un passaggio di convalida, oppure una decisione documentata per i casi rimanenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The style guide mentiones presentation use case for the course. That means much inconvenience while dealing with very long lines in the code snippets.
The problem was addressed earlier as it can be seen from rustfmt.toml configuration:
# The code blocks get a scrollbar if they are wider than this.
max_width = 85
This works well for read only blocks, but fails for editable ones. Due to line numbering, there is less space left for the code. My observations are that the scrolling threshold is 83 for less than 10 line long code, 82 - less than 100 and 81 - above 100. There are no snippets above 1k and I suppose there must not be in the context of the course.
The following merge request adopts max 81-character width and eliminates detected scrolling in rust code blocks.
I suppose it is a partial solution though. But further steps require a discussion.
There should be an automated format validation step. rustfmt can do so with rust files, but not markdown. Across the course there are examples where rust code is separated from markdown files and included like:
{{#include testing/src/lib.rs:leftpad}}
But. Firstly, that means all the rust code should be excluded from md files and guides updated. Secondly, rust code contains comments that are translated. And translations can turn out to be longer than the source. And the result should also be tested. Thirdly, rustfmt seems to ignore long use expressions and long strings in macro arguments - leaves them as they are without errors.
Also the course has some code blocks with long shell commands and text output. Were left unchanged.
Summary questions:
- Should we exclude all rust code from markdown files?
- How can we validate rust code with translated comments?
- How to deal with
useand macros arguments that can still produce horizontal scrolling? - What to do with other code blocks with very long lines?
- Lingua principale
- Rust
- Stelle
- 33.4k
- Fork
- 2.1k
- Merge medio
- 2g 9h
- PR unite (30g)
- 16
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di google/comprehensive-rust
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
google/comprehensive-rust#2955 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
google/comprehensive-rust#2733 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
google/comprehensive-rust#2732 ·
I maintainer di solito rispondono entro 1 giorno
-
Minor suggested updates to the Collatz Sequence exercise in section 6.8 and its solution in section 6.8.1Forse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
google/comprehensive-rust#2636 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
google/comprehensive-rust#3155 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di google/comprehensive-rust
Issue simili
-
[Feature] 设置里面的同步功能Apertaenhancement user-priority/P2
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 1 giorno
-
agent:triaged bug bughunt pm:npm priority:p1
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
SocketDev/socket-patch#1127 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno