Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Native browser find (`Ctrl/Cmd+F`) does not search the full diff in diff view

Abierto
#12 0 comentarios 3 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
5/5
Tiempo estimado
Más de una semana
Aptitud para principiantes
45/100
Tipo de issue
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Tranquilo
Stack tecnológico
typescript
Área
frontend, web-dev

Línea de trabajo

El issue no menciona archivos ni tests, así que empieza siguiendo la implementación de la vista de diff y sus rutas de virtualización, secciones contraídas, vista previa y Load diff. Compara el comportamiento híbrido propuesto y define tests para la búsqueda nativa en secciones ocultas y renderizadas; se considera terminado cuando las coincidencias se pueden buscar sin eliminar las características de rendimiento de los diffs grandes.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Problem

In diff view, native browser find only searches content that is currently mounted in the DOM. Today that means Ctrl/Cmd+F can miss valid matches because parts of the diff are intentionally hidden or not rendered:

  • Virtualized file blocks outside the viewport are not mounted.
  • Auto-collapsed files hide their diff content.
  • Rendered previews replace source diff text for renderable files.
  • Large diffs may show a Load diff placeholder instead of line content.
  • Omitted context is not present until expanded.

This makes browser find unreliable for reviewing large or mixed-content diffs.

Proposals

  • Add a temporary “browser find mode” on Ctrl/Cmd+F that fully renders the diff, expands hidden sections, and forces source view so native browser find can search the whole diff.
  • Make the normal diff mode always fully searchable by keeping all source diff text mounted in the DOM, even when sections are collapsed or previewed.
  • Build an app-level search experience instead of relying on native browser find.

Recommendation

Use a hybrid approach:

  • For small diffs, keep the normal mode fully searchable without a separate mode.
  • For large diffs, enter a temporary browser-find mode on Ctrl/Cmd+F that fully renders the diff and forces source text visible.

This keeps the common case simple while avoiding the steady-state performance and UI cost of fully rendering large diffs all the time.

Notes

  • Native browser find cannot tell us which hidden section matched, so lazy “expand only the matching file” behavior is not practical.
  • The hybrid approach preserves current performance characteristics for large diffs while improving searchability for most day-to-day reviews.
Image
Lenguaje dominante
TypeScript
Estrellas
775
Forks
57
Métricas de merge de PR
Sin PR fusionados en 30 d

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de nilbuild/diffity

Todos los issues de nilbuild/diffity

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.