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

fix: added condition to set the listing image caption #851

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

Wagner3UB
Copy link

@Wagner3UB Wagner3UB commented Jan 7, 2025

US: https://redturtle.tpondemand.com/entity/62740-in-tutti-i-template-del-blocco

Aggiunta logica per la corretta assegnazione degli attributi alt e title nelle immagini dei blocchi listing

  • Implementata una nuova condizione per popolare correttamente l'attributo alt nelle immagini di anteprima o testata.
  • Aggiunta la verifica per garantire che l'attributo title venga compilato solo se esiste una didascalia associata all'immagine di anteprima o testata. È importante evidenziare che il title non verra compilato mai da solo, neanche se il campo alt sia vuoto.
  • Rimosso l'attributo showTitleAttr dalle card nei blocchi listing, adesso questo attributo viene generato dentro il componente ListingImage.

È importante mantenere le richieste anteriore create per Parma - l'immagine non può contenere title="" se alt="" è vuoto:
us: https://redturtle.tpondemand.com/entity/50642-rimuovere-title-da-immagine-della-card
pr: #558

@Wagner3UB Wagner3UB self-assigned this Jan 7, 2025
@Wagner3UB Wagner3UB added the a11y-accessibility Fixes accessibility issues or adds accessibility features label Jan 9, 2025
@Wagner3UB
Copy link
Author

Attenzione: il metadata image_caption ancora non viene passato dal BE correttamente. Sono in attesa per fare i test finale.

Copy link

@giuliaghisini giuliaghisini left a comment

Choose a reason for hiding this comment

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

sistemato questo per me è ok

RELEASE.md Outdated Show resolved Hide resolved
Co-authored-by: Giulia Ghisini <[email protected]>
@pnicolli
Copy link
Member

pnicolli commented Feb 6, 2025

Lascio una nota aggiuntiva: attenzione a conflitti ma anche a fare una nuova ricerca per showTitleAttr perché in quest'altra PR ne sono stati aggiunti alcuni: #861

@Wagner3UB
Copy link
Author

Lascio una nota aggiuntiva: attenzione a conflitti ma anche a fare una nuova ricerca per showTitleAttr perché in quest'altra PR ne sono stati aggiunti alcuni: #861

Ok, verifico appena riesco. Grazie.

@Wagner3UB Wagner3UB requested a review from pnicolli February 7, 2025 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y-accessibility Fixes accessibility issues or adds accessibility features porting-needed rer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants