Marking a file as Viewed should scroll to the next unviewed file
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 68/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- frontend
Direzione di ricerca
Inizia leggendo la funzione pura nextUnviewedFile e il relativo unit test per comprendere la selezione in base all’ordine di visualizzazione. Segui il punto in cui contrassegnare un file come Viewed aggiorna la Review UI, quindi assicurati che il viewport si sposti all’inizio del file successivo non visualizzato, senza ricominciare dall’inizio né spostarsi quando il contrassegno di un file viene rimosso; l’unit test esistente dovrebbe coprire il comportamento di selezione.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Summary
In a large diff, marking a file Viewed leaves the viewport on the collapsed file. The reviewer then scrolls manually to find the next file that still needs review. GitHub's review UI advances to the next unviewed file.
Proposal
When a file is marked Viewed, scroll to the top of the next unviewed file in display order. No scroll occurs when no unviewed file follows (the search does not wrap to earlier files), or when a file is un-marked.
Notes
The selection logic is a pure function (nextUnviewedFile) with a unit test. A PR follows.
- Lingua principale
- TypeScript
- Stelle
- 208
- Fork
- 38
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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 wong2/diffx
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 38/100
Issue simili
-
bug cli service
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
create-element: same editorAlias silent-fallback bug as #201, not covered by that fixForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertagenerated-by-ai
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
umbraco/Umbraco-CMS-MCP-Editor#208 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Table: Space fires onActivate in single-selection mode — the reference doc and the JSDoc disagreeAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
sidorares/react-x11-components#764 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
backnotprop/plannotator#1840 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
JoviDeCroock/pracht#432 ·
I maintainer di solito rispondono entro 1 giorno