refactor: consolidate 12 near-identical SVG icon components into one pattern
Los mantenedores suelen responder en 1 día
@Talha12Shiekh ya está trabajando en esto.
Desde el 28/7/2026.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- javascript, react
- Área
- frontend
Línea de trabajo
Compara SearchSvg.js, CancelSvg.js, ThumbsUpSvg.js, ThumbsDownSvg.js, MicSvg.js, MuteSvg.js, ListenSvg.js, AutofillSvg.js, DownloadSvg.js, CustomSvg.js e Icons.js, empezando por su wrapper compartido, propTypes y estructura de componentes. Decide entre los enfoques de registry y sprite, y después verifica que todos los iconos existentes sigan disponibles sin boilerplate duplicado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
The shared components directory has 12 individual SVG components, each with:
- The same styled wrapper import (
@emotion/styled) - The same propTypes declaration
- The same React component structure
- Only the SVG
<path>data differs
Files: SearchSvg.js, CancelSvg.js, ThumbsUpSvg.js, ThumbsDownSvg.js, MicSvg.js, MuteSvg.js, ListenSvg.js (523 lines!), AutofillSvg.js, DownloadSvg.js, CustomSvg.js, Icons.js (has 4 more icons)
~83% of each file is duplicated boilerplate.
Suggested Fix
Option A: A single icons.js registry mapping icon names to SVG path data, with one shared Icon component.
Option B: A sprite-based approach using <symbol> definitions with <use> references.
Severity
Medium — codebase bloat, ~600 lines of duplication
- Lenguaje dominante
- JavaScript
- Estrellas
- 4.9k
- Forks
- 479
- Merge medio
- 4 d 10 h
- PR fusionados (30 d)
- 4
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 appbaseio/reactivesearch
-
ci: add Node.js version matrix, caching, and lint step to test workflowPosiblemente ocupada @Talha12Shiekh la tomó hace 69 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
appbaseio/reactivesearch#2322 ·
Los mantenedores suelen responder en 1 día
-
chore: add Dependabot config for automated dependency updatesPosiblemente ocupada @Talha12Shiekh la tomó hace 69 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
appbaseio/reactivesearch#2321 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
security: user-controlled URL params injected directly into Redux store via JSON.parsePosiblemente ocupada @Talha12Shiekh la tomó hace 69 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
appbaseio/reactivesearch#2319 ·
Los mantenedores suelen responder en 1 día
-
refactor: SearchBox.js is 2692 lines — extract subcomponents for maintainabilityPosiblemente ocupada @Talha12Shiekh la tomó hace 69 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
appbaseio/reactivesearch#2323 ·
Los mantenedores suelen responder en 1 día
-
bug: image src attributes not validated for javascript: protocol URLsPosiblemente ocupada @Talha12Shiekh la tomó hace 69 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
appbaseio/reactivesearch#2320 ·
Los mantenedores suelen responder en 1 día
Todos los issues de appbaseio/reactivesearch
Issues similares
-
enhancement
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
-
app bug config windows-os
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
openai/codex#51926 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
clawsweeper:needs-product-decision clawsweeper:needs-security-review clawsweeper:no-new-fix-pr clawsweeper:source-repro impact:security issue-rating: 🦞 diamond lobster P2
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
openclaw/openclaw#166870 · 2 comentarios · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
⚠ needs intervention document structure changed
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
[Package] Widget primitives [Type] Enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
WordPress/gutenberg#84204 · 1 comentario ·
Los mantenedores suelen responder en 1 día