Marking a file as Viewed should scroll to the next unviewed file
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- frontend
Línea de trabajo
Empieza leyendo la función pura nextUnviewedFile y su prueba unitaria para entender la selección según el orden de visualización. Sigue dónde marcar un archivo como Viewed actualiza la Review UI y, después, asegúrate de que el viewport se mueva al principio del siguiente archivo no visto, sin volver al principio ni moverse cuando se quite la marca de un archivo; la prueba unitaria existente debería cubrir el comportamiento de selección.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- TypeScript
- Estrellas
- 206
- Forks
- 37
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de wong2/diffx
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 38/100
Todos los issues de wong2/diffx
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
lichess-org/api#678 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
PostHog/posthog.com#20628 ·
Los mantenedores suelen responder en 1 día
-
bug status:Needs Triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
jupyterlab/jupyterlab#19964 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
agentscope-ai/QwenPaw#8064 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
area: notebooks-jupyter bug theme: new notebook frontend
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
posit-dev/positron#16347 · 1 comentario ·
Los mantenedores suelen responder en 1 día