Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Necesitamos Antd? #41

Open
Javier-Rotelli opened this issue Sep 30, 2019 · 3 comments
Open

Necesitamos Antd? #41

Javier-Rotelli opened this issue Sep 30, 2019 · 3 comments
Labels

Comments

@Javier-Rotelli
Copy link
Member

aloja! aca vengo a polemiquear.
realmente necesitamos un framework complejo como antd? en mi experiencia a la larga se pierde mas tiempo luchando contra una herramienta asi, que escribiendo los comonentes que necesitamos a mano.

@fdemian vos sos el que lo metio. me gustaria escuchar tu opinion

@fdemian
Copy link
Contributor

fdemian commented Oct 8, 2019

En principio estaría opuesto a cambiarlo.

En mi experiencia lo use para casi todos los proyectos nuevos que tengo en React y te maneja muchas cosas copadas con los componentes que tiene. Quizas no aplica inmediatamente para el unico componente que tenemos en el proyecto. Mas que nada lo que aporta es styling copado + funcionalidad (los componentes que usa estan disponibles sin styling como rc-<nombre del componente> y en ultima instancia se pueden usar separados del styling).

Si usamos bien cosas como code splitting en react, no hay necesidad de que notemos el tamaño de la libreria en cuanto a lo que pide el usuario desde el servidor (hice pruebas en otro proyectos contra el speed rank de google y es posible con retoques minimos que llegar a que te de todo en verde).

Amen de eso, me gustaría ver qué alternativas tenemos que estén al mismo nivel antes de sacarlo. Sacarlo para dejar la applicación pelada de estilos no me convence para nada.

Si bien podemos hacer la UI a mano y darle estilo a los botones/componentes/etc, me parece que eso ya requiere que tengamos a alguien dedicado a hacer eso y que tenga un mínimo diseño viable antes de proceder (la layout la podemos manejar con flexbox, pero esto ya tiene algo similar, por ej.).

@Javier-Rotelli
Copy link
Member Author

diseño vamos a necesitar. antd o no. creia que eso ya lo habiamos acordado... eventualmente el styling deberia ser todo nuestro.
que funcionalidad sentis que aporta que no se pueda negociar?

@fdemian
Copy link
Contributor

fdemian commented Oct 19, 2019

¿Features no negociables? No se, podemos encontrar un reemplazo para casi todo.
No me parece bien sacar el styling para dejar la aplicación pelada, pero si tuvieramos una libreria de UI alternativa o styling propio decente ya implementado (y en un PR) no me parecería mal hacer el cambio.

Acá es donde saco a colación un punto pragmático: todavía no tenemos diseño propio y venimos con ese tema desde hace un año (más de un año si recuerdo bien). Esto va a sonar pesimista, pero no creo que a fin de año vayamos a tener styling propio. Al menos de mi lado, el styling no es mi especialidad (mas allá de meterle align:center; al texto de un botón 😄 ) .

¿Tu única objeción a tener una librería de UI es que es pesada? Porque me parece que lo único que impacta de eso es el tiempo de carga de la página del lado del usuario. No es algo que ahora mismo sea muy dificil de reducir si usamos bien code splitting y demás yerbas. Si te parece que es un problema muy apremiante lo medimos y vemos que tanto impacta antd en los tiempos de carga.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants