Avatar: getAvatarColor collides on anagrams; add color seed + palette subset
Los mantenedores suelen responder en 2 días
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 52/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- react, typescript
Línea de trabajo
Comienza leyendo components/avatar/utils.tsx y siguiendo la ruta de color automático de Avatar. Confirma las decisiones de la API para una semilla de color opcional y un subconjunto de la paleta; después, verifica que el hash distinga los cambios de orden y conserve el comportamiento de fallback. El trabajo estará terminado cuando los colores automáticos admitan semillas independientes estables y paletas restringidas sin afectar al trabajo sobre colores explícitos descrito en #650.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
getAvatarColor picks an avatar color from a name using a char-code sum, which collides on anagrams and clusters on similar strings. There is also no way to seed the color independently of the display name, and no way to restrict the palette to a subset. Consumers with a specific color language end up re-implementing the whole function.
The hashing bug
components/avatar/utils.tsx:
export function getAvatarColor(str: string): AVATAR_COLORS {
const hash = str.split('').reduce((acc, char) => acc + char.charCodeAt(0), 0);
const index = hash % COLORS.length;
return COLORS[index];
}
Summing char codes is order-insensitive, so any anagram maps to the same color — "Ravi" and "Vira", "Amy Lee" and "Ely Mae" collide. Names that differ by one adjacent-letter swap also collide, and sequences like "User 1" / "User 2" / "User 3" land on adjacent (often visually close) palette entries instead of spreading out.
A small change fixes the distribution — a polynomial (order-sensitive) hash:
let hash = 0;
for (let i = 0; i < str.length; i++) {
hash = (hash * 31 + str.charCodeAt(i)) | 0;
}
const index = Math.abs(hash) % COLORS.length;
Two API gaps
1. No color seed. The color is derived from the string passed in — which is the display name — so a rename changes a person's color, and two people with the same name always share one. A separate, stable seed (e.g. an id) would keep colors stable across renames and distinct for namesakes. Suggest an optional colorSeed/seed prop on Avatar (falling back to the name today).
2. No palette subset. COLORS is a fixed 13-entry list including warm tones (orange, crimson, gold) and neutral. A product with a cooler or brand-restricted identity language can't say "only pick from these." Suggest letting getAvatarColor (and Avatar's auto-color) accept an optional palette subset.
Relationship to #650
#650 (item 3) proposes propagating an explicit color prop to Avatar. This issue is about the automatic color path — hash quality and seeding — which is orthogonal: even with an explicit-color prop, the auto-assigned fallback should distribute well and stay stable.
Willing to contribute
Happy to open a PR — the hash change alone is a one-liner, and the seed/palette props are additive.
- Lenguaje dominante
- TypeScript
- Estrellas
- 69
- Forks
- 13
- Merge medio
- 3 d 15 h
- PR fusionados (30 d)
- 24
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 raystack/apsara
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 2 días
-
DataView: don't offer select/multiselect filters with empty filterOptionsPosiblemente ocupada @rohanchkrabrty la tomó hace 6 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 2 días
Todos los issues de raystack/apsara
Issues similares
-
Remove the landing pageAbiertoby: ai-assisted frontend good-for: new-member spike
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Northeastern-Electric-Racing/Argos#847 ·
Los mantenedores suelen responder en 4 días
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 84/100
SignalK/freeboard-sk#990 ·
Los mantenedores suelen responder en 1 día
-
[missing-inheritance] audit review (1 preset)Posiblemente ocupada @github-actions la tomó hoy. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
osmberlin/tagging-schema-browser#363 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Albert-Weasker/niubigeo#205 ·
Los mantenedores suelen responder en 1 día
-
area/frontend area/v2 kind/bug priority/needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
kubeflow/notebooks#1498 · 1 comentario ·
Los mantenedores suelen responder en 1 día