showInfobar setting is ignored (config reads "showInfo" instead)
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 88/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- frontend
Línea de trabajo
Empieza con src/config.ts e inspecciona sanitizeConfig() y la migración del info heredado; el issue identifica ambas claves incorrectas e indica el comportamiento sugerido. Ejecuta los casos reportados de sanitizeConfig() y fromQuery() para JSON, URL, valores heredados y valores predeterminados no establecidos. El trabajo está terminado cuando cada caso conserva la configuración esperada para mostrar u ocultar, incluidas las configuraciones que usan la solución provisional showInfo.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
"showInfobar": false in wall-config.json (or a ?load= config) has no effect; the info bar is always shown. The same happens with ?info=false in the URL and the "Show info bar" checkbox in the Customize dialog.
Cause
In src/config.ts, sanitizeConfig() reads the wrong key:
result.showInfobar = boolOr(config.showInfo, fallback.showInfobar)
config.showInfo is never set (the type, the URL parser and the generated config all use showInfobar), so it always falls back to the default (true).
A related typo in the legacy migration sets config.showinfo (lowercase), so the old info: "top"/"hide" values are ignored too.
Workaround for now: use "showInfo": false in the JSON config.
Suggested fix
if (isString(config.info))
- config.showinfo = config.info == "top"
+ config.showInfobar ??= config.info == "top"
...
- result.showInfobar = boolOr(config.showInfo, fallback.showInfobar)
+ result.showInfobar = boolOr(config.showInfobar ?? config.showInfo, fallback.showInfobar)
Keeping ?? config.showInfo means configs that already use the workaround keep working.
Tested by running sanitizeConfig() / fromQuery() from current main (b55d81c) with Node: before the fix, {showInfobar: false} and info=false both give true; after it, every case (JSON true/false, URL true/false, legacy "top"/"hide", unset → default) gives the expected value.
- Lenguaje dominante
- TypeScript
- Estrellas
- 96
- Forks
- 11
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin 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 defnull/fediwall
-
just saying hiAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 1/100
-
bird.makeupAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 38/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 39/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
Todos los issues de defnull/fediwall
Issues similares
-
account bug communication lecture
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
area:tui bug platform:macos
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
anthropics/claude-code#109005 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 87/100
openedx/end-to-end-tests#196 ·
Los mantenedores suelen responder en 1 día
-
area:tooling good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
interledger/publisher-tools#1007 ·
Los mantenedores suelen responder en 1 día
-
priority/medium roadmap/v1.1.1 stakeholder/rapr type/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
databrickslabs/ontos#1420 ·
Los mantenedores suelen responder en 3 días