add UI feedback after saving a routine

Abierto
#1,114 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
45/100
Tipo de issue
Nueva funcionalidad
Claridad
Bien especificado
Estado de actividad
Estancado
Stack tecnológico
react, typescript
Área
frontend

Línea de trabajo

Start from the routine edit screen where the "Save" button lives (the before/after screenshots show the routine form); trace the save mutation/handler to see where success or failure is already known. Check the codebase for an existing toast or notification convention before adding a new reusable button-state component. Done looks like: clicking Save briefly renders a ✅ on success or ❎ on failure, then reverts to the normal label. Note the issue is 385 days old with zero comments and no reply on Discord, so confirm a maintainer still wants it before investing.

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

Descripción

Hi, I noticed that when editing a routine and hitting "Save", the data actually gets saved (after checked), but there’s no clear feedback in the UI.
I didn’t see any success message, so it feels a bit confusing — I wasn’t sure if the changes went through.
(I also tried asking on Discord but didn’t get a reply yet )

Before saving:
Image

After saving:
Image

Here’s an idea:
after clicking save, the button could briefly show ❎ if it fails or ✅ if it succeeds, then switch back to the normal "Save" button.

This could probably be made as a reusable component.

Lenguaje dominante
TypeScript
Estrellas
37
Forks
94
Merge medio
15 h 10 min
PR fusionados (30 d)
14

Preparar el entorno

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 wger-project/react

Todos los issues de wger-project/react

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.