Button should have an override to prevent from going to Icon in mobile
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 58/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- react, typescript
Línea de trabajo
Empieza en Button.tsx, en la rama isXSmall que cambia entre IconButton y StyledButton. Revisa cómo los botones Edit y Save pasan las props y, después, añade el override solicitado preservando el comportamiento predeterminado actual. Estará terminado cuando los consumidores puedan mantener estos botones a ancho completo en dispositivos móviles sin implementar botones personalizados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Is your feature request related to a problem? Please describe.
We have what I thought would be a fairly normal scenario where our designs require the actions Buttons to remain full width in mobile and desktop. But when going to mobile they default to an icon button and there is no way to override this.
Describe the solution you'd like
A prop that can be passed to an Edit, Save button that simply overrides the isXSmall condition on a button.
Describe alternatives you've considered
Having to implement my own Edit or Save buttons and then removing that functionality however this seems wasteful.
Additional context
In Button.tsx we could just have an option added such as shrink. Which can default to true.
return isXSmall && shrink ? (
label && !disabled ? (
<Tooltip title={translatedLabel}>
<IconButton
aria-label={translatedLabel}
className={className}
color={color}
size="large"
{...rest}
{...linkParams}
>
{children}
</IconButton>
</Tooltip>
) : (
<IconButton
className={className}
color={color}
disabled={disabled}
size="large"
{...rest}
{...linkParams}
>
{children}
</IconButton>
)
) : (
<StyledButton
className={className}
color={color}
size={size}
aria-label={translatedLabel}
disabled={disabled}
startIcon={alignIcon === 'left' && children ? children : undefined}
endIcon={alignIcon === 'right' && children ? children : undefined}
{...rest}
{...linkParams}
>
{translatedLabel}
</StyledButton>
);
If you're happy with this then i'd be happy to open a PR. Is there anymore use cases that I'm potentially missing?
- Lenguaje dominante
- TypeScript
- Estrellas
- 27k
- Forks
- 5.5k
- Merge medio
- 1 d 3 min
- PR fusionados (30 d)
- 32
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 marmelab/react-admin
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
marmelab/react-admin#11431 ·
Los mantenedores suelen responder en 1 día
-
Pagination: page button aria-labels are not translated on small screensPosiblemente ocupada @Singmin la tomó hace 2 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
marmelab/react-admin#11427 ·
Los mantenedores suelen responder en 1 día
-
Add emptyText property to SelectArrayInputPosiblemente ocupada @kwitykwity la tomó hace 60 días. Abiertoenhancement good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
marmelab/react-admin#10454 · 7 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 Más de una semana Aptitud para principiantes 38/100
marmelab/react-admin#11426 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
ra-data-graphql-simple: outdated graphql-ast-types-browser dependency causes npm peer warningsQuizá libre de nuevo @bipul724 la tomó hace 14 días y no hay ningún pull request abierto. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
marmelab/react-admin#11395 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de marmelab/react-admin
Issues similares
-
area:cli bug P4
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
bug from-studio
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
esengine/DeepSeek-Reasonix#14868 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
loopbackio/loopback-next#11878 ·
Los mantenedores suelen responder en 2 días
-
bug 🐛
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
comunica/comunica#1925 · 1 comentario ·
Los mantenedores suelen responder en 1 día