intersystems / intersystems/git-source-control
Git Web UI: Merge output quickly becomes unreadable
- Lingua principale
- ObjectScript
- Stelle
- 22
- Fork
- 14
- Merge medio
- 1g 9h
- PR unite (30g)
- 5
Descrizione
I used the Git Web UI to pull down changes to a branch (checked out `main`, synced remote branches, merged `origin/main` into `main`). The resulting merge commit contained 7 new and 1 changed files (git output was 18 lines total), plus the corresponding "Input" actions logging which had 24 lines (for a total of 42 lines of output). There were new-lines within the text, but all of these were mashed together into a single line in the HTML view (because HTML does not recognize line breaks on their own)
This makes the output log extremely difficult for a human to visually skim. (My browser was maximized and the alert message box took up about 30%-40% of the vertical screen height.) In my experience, pulling down commit histories containing 3+ files is the norm.
A quick solution would be to place all this text inside a `
` element and give it a reasonable `max-height` (say 200-ish px) so large pulls/merges do not cause the alert notification region to completely overtake the view screen. A better solution is probably to actually parse lines into an HTML equivalent (``, `
`) so you get text wrapping too.
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia dal percorso di Git Web UI che visualizza l'output di merge e pull, quindi riproduci un merge che coinvolga più file in modo che l'output includa più righe. Il lavoro è completato quando le interruzioni di riga rimangono leggibili nell'HTML e un output di pull o merge di grandi dimensioni non fa sì che l'area degli avvisi occupi tutto lo schermo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- git
- Ambito
- web-dev
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 45/100