Skip to content

Improve Job Post and Blog Image Formatting#68

Open
aog-frogtek wants to merge 7 commits intomasterfrom
blog/visual-and-format-fixes-v2
Open

Improve Job Post and Blog Image Formatting#68
aog-frogtek wants to merge 7 commits intomasterfrom
blog/visual-and-format-fixes-v2

Conversation

@aog-frogtek
Copy link
Collaborator

#64

@aog-frogtek aog-frogtek requested a review from durey February 17, 2025 14:21
@aog-frogtek
Copy link
Collaborator Author

@durey he hecho estos cambios al código de Ruben:

  • Él modificaba el gitignore para no guardar el archivo Gemfile.lock pero he visto que esta es la forma de actualizar librerías así que he revertido ese cambio.
  • He respetado los cambios en _includes/post_except.html ya que hacen lo que se pide en los ATs de la tarjeta (poner negritas).
  • He cambiado los posts mal codificados (2 espacios al final de las líneas de rol, industria, etc) y he hecho lo mismo con los últimos que no estaban en la PR.
  • He respetado los cambios en assets/css/main.scss para hacer un resize de las imágenes si la ventana cambia de tamaño (lo pone en los ATs).

@aog-frogtek
Copy link
Collaborator Author

Copy link
Member

@durey durey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aog-frogtek y si ponemos una línea en blanco en lugar de dos espacios en blanco al final, ¿hace el mismo efecto luego al renderizar? pregunto por si eso soluciona el problema de los editores de texto que hacen un TRIM al guardar

@durey
Copy link
Member

durey commented Feb 18, 2025

@aog-frogtek habría que fusionar todos los commits en uno y luego separarlos bien

@aog-frogtek
Copy link
Collaborator Author

@aog-frogtek y si ponemos una línea en blanco en lugar de dos espacios en blanco al final, ¿hace el mismo efecto luego al renderizar? pregunto por si eso soluciona el problema de los editores de texto que hacen un TRIM al guardar

Si pongo una línea en blanco ya no aparece en el resumen (no funciona vaya)..

@aog-frogtek aog-frogtek force-pushed the blog/visual-and-format-fixes-v2 branch from f573256 to 036a422 Compare February 18, 2025 11:04
@aog-frogtek aog-frogtek requested a review from durey February 18, 2025 11:06
@aog-frogtek
Copy link
Collaborator Author

aog-frogtek commented Feb 18, 2025

Ya junté todo en un commit @durey . No entiendo lo de "separarlos bien".

@durey durey force-pushed the blog/visual-and-format-fixes-v2 branch from 036a422 to 2fe864f Compare February 18, 2025 11:30
@durey
Copy link
Member

durey commented Feb 18, 2025

@aog-frogtek ,

  • Ya separé los commits según la funcionalidad que implementan, así te haces una idea.
  • Aproveché para actualizar todas las librerías a la última versión.
  • Ojo, el resumen del blog ahora mete unos logos que no estaban antes (fíjate por ejemplo en "El Gobierno de Aragón y los fondos FEDER apoyan los proyectos de Frogtek también en 2023" desde la pantalla del resumen de artículos del blog).

@aog-frogtek
Copy link
Collaborator Author

@aog-frogtek ,

  • Ya separé los commits según la funcionalidad que implementan, así te haces una idea.
  • Aproveché para actualizar todas las librerías a la última versión.
  • Ojo, el resumen del blog ahora mete unos logos que no estaban antes (fíjate por ejemplo en "El Gobierno de Aragón y los fondos FEDER apoyan los proyectos de Frogtek también en 2023" desde la pantalla del resumen de artículos del blog).

He corregido ese comportamiento y he cambiado también dónde se asignan los estilos para el tema de las imágenes:

  • Ya no modifico el archivo main.scss
  • Añado un tag de style a la plantilla de post para que las imágenes tengan resize según el tamaño de la ventana.
  • Añado un tag de style a la plantilla de post_excerpt (el resumen) para que los resúmenes no muestren imágenes.
  • También he cambiado la lógica del excerpt-content (en el post_excerpt) ya que no le veía sentido a ese IF.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants