doofinder / doofinder-magento2

Open Software License 3.0
10 stars 9 forks source link

[Magento] Reestablecer opción de seleccionar tamaño de imagen #329

Closed sofia-doofinder closed 2 months ago

sofia-doofinder commented 3 months ago

Notion: https://www.notion.so/Magento-resize-images-1361222f883a4739ad0659e07eca5f9a

ogomezba commented 3 months ago

@IgnacioPursalsZ Tras comentar con Ignacio diferentes puntos de la solución:

sofia-doofinder commented 3 months ago

@ogomezba un par de preguntas:

ogomezba commented 3 months ago

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:

ogomezba commented 3 months ago

@ogomezba un par de preguntas:

  • Qué es lo que se va a poner en el selector: "Base, Small, Thumbnail, Swatch" o "470x470, 370x370, 100x100, 50x50" o qué exactamente?
  • No entiendo en el último punto cuando dices que se va a indexar la marcada con el tag "small", que es que vamos a coger esa y la vamos a redimensionar? o por qué no cogemos directamente el tamaño que tienen marcado?

@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.

IgnacioPursalsZ commented 3 months ago

Lo mejor para estos campos sería como están definidos en la tarjeta:

ogomezba commented 3 months ago

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.