sam-upload-v2 don't update entire table ngOnChanges
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 55/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- angular, typescript
- Área
- frontend
Línea de trabajo
Comienza en src/ui-kit/form-controls/upload-v2/upload-v2.component.ts, en ngOnChanges, especialmente alrededor de la línea 253, y sigue cómo los cambios en uploadedFiles reconstruyen la tabla. Comprueba cómo se representan los controles de archivos individuales e identifica el comportamiento existente para los archivos modificados o añadidos. Se considera terminado cuando los controles que no han cambiado, incluido su estado de alternancia, se conservan y solo se recrean los controles modificados o nuevos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
sam-upload-v2 component in ngOnChanges is recreating the entire table whenever there is any change to the uploadedFiles input (https://github.com/GSA/sam-ui-elements/blob/master/src/ui-kit/form-controls/upload-v2/upload-v2.component.ts#L253).
When this happens, it causes a very noticeable visual refresh of the toggle switches. If a toggle switch is on, after ngOnChanges it will be replaced with a new switch that defaults to off, which will then animate itself turning on again.
To prevent this, could we go one level more granular in ngOnChanges and test whether there were any changes to each file control in the table, and only remake those that changed or were added?
- Lenguaje dominante
- TypeScript
- Estrellas
- 8
- Forks
- 34
- Merge medio
- 2 d 14 h
- PR fusionados (30 d)
- 5
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
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 GSA/sam-ui-elements
-
github_actions security
Dificultad 2/5 Medio día Aptitud para principiantes 88/100
GSA/sam-ui-elements#761 ·
Los mantenedores suelen responder en 1 día
-
tech-debt
Dificultad 4/5 3-5 días Aptitud para principiantes 56/100
GSA/sam-ui-elements#708 ·
Los mantenedores suelen responder en 1 día
-
disablePageTabIndex() sets aria-hidden on the focused calendar button, and enablePageTabIndex() never removes itQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abiertobugfix
Dificultad 4/5 3-5 días Aptitud para principiantes 68/100
GSA/sam-ui-elements#682 ·
Los mantenedores suelen responder en 1 día
-
bugfix
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
GSA/sam-ui-elements#681 ·
Los mantenedores suelen responder en 1 día
-
Burn down ESLint and Angular lint debtQuizá libre de nuevo @fpigeonjr la tomó hace 63 días y no hay ningún pull request abierto. Abiertoangular-upgrade maintenance
GSA/sam-ui-elements#580 · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de GSA/sam-ui-elements
Issues similares
-
area:plugins area:security area:skills bug has repro
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
anthropics/claude-code#105911 ·
Los mantenedores suelen responder en 1 día
-
openspec-apply-change skill: when blocked, may select a ready artifact unrelated to missingArtifactsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Fission-AI/OpenSpec#2165 ·
Los mantenedores suelen responder en 1 día
-
🐞 bug 🧐 unconfirmed
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Tencent/tdesign-vue-next#7032 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
gnosis/gnosis_vpn#661 ·
Los mantenedores suelen responder en 1 día
-
content-freshness: staleOwnedContent doc comments still say 'complete receipt' (follow-up to #7912)Abiertotriage:bug-verified
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
Los mantenedores suelen responder en 1 día