-
Notifications
You must be signed in to change notification settings - Fork 8
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
[Magento] Reestablecer opción de seleccionar tamaño de imagen #329
Comments
@IgnacioPursalsZ Tras comentar con Ignacio diferentes puntos de la solución:
|
@ogomezba un par de preguntas:
|
Solución técnica discutida con @eduardogomez97 . La solución consistirá en la adición de una nueva picklist en la configuración del plugin que permitirá seleccionar un tamaño. Dicha selección se guardará en la base de datos de Magento tal y como se hace para el resto de configuraciones. Dicha configuración se utilizará en el momento de la indexación para generar la imagen correctamente. Puntos adicionales importantes:
|
@sofia-doofinder A mi parecer, creo que sería menos confuso poner los tamaños (470x470, etc.). @IgnacioPursalsZ qué valores deberías mostrar en la picklist? Respecto a lo de "small", ahora mismo siempre indexamos la imagen marcada como "small" y en tamaño original. Discutiendo con @IgnacioPursalsZ, hemos acordado que tenía sentido seguir eligiendo esa imagen (para no cambiar la imagen que ya se está indexando) y redimensionarla al tamaño elegido por el usuario. A priori sería más confuso si, adicionalmente del tamaño, cambiamos también la imagen indexada. |
Lo mejor para estos campos sería como están definidos en la tarjeta:
|
Tras investigar un poco más, el comportamiento actual no es de seleccionar siempre la imagen con el tag "small", sino elegir la primera imagen de la list que tenga, o bien el tag small o bien thumbnail. Tras comentarlo con Ignacio, se decide mantener este comportamiento aunque se incluya el resize. |
Notion: https://www.notion.so/Magento-resize-images-1361222f883a4739ad0659e07eca5f9a
The text was updated successfully, but these errors were encountered: