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

Appearance card in slider template #512

Merged
merged 18 commits into from
Apr 8, 2024
Merged

Conversation

giuliaghisini
Copy link

Nel template 'Slider' del blocco elenco, ora è possibile scegliere un aspetto diverso per gli elementi dello slider.
Il default è quello che c'è sempre stato, in più si può scegliere di mostrare gli elementi nel formato 'Card semplice' o nel formato 'Card con immagine' .
E' anche facile aggiungere nuovi template custom dal config.blocks.listing.variations e modificando la variazione 'slider'

_Registrazione.schermo.2024-02-01.alle.14.48.49.mov

Copy link
Member

@pnicolli pnicolli left a comment

Choose a reason for hiding this comment

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

Io farei in modo che le card siano tutte alte uguali, quando vengono usate per i listing normali lo fanno e mi sembra migliore. Per il resto mi pare ok.

@giuliaghisini
Copy link
Author

@pnicolli ho anche aggiunto un controllo sul numero di slide da mostrare nel caso in cui questo fosse maggiore del numero di elementi risultanti dalla query, perchè in quel caso si vedevano gli elementi ripetuti nello slider e lo slider si vedeva male

@giuliaghisini
Copy link
Author

@pnicolli ho anche splittato il file degli stili del template cardWithImage e inEvidenceTemplate

…ard_in_slider_template' of github.com:RedTurtle/design-comuni-plone-theme into appearance_card_in_slider_template
@Wagner3UB Wagner3UB self-requested a review February 20, 2024 09:13
@giuliaghisini
Copy link
Author

giuliaghisini commented Feb 22, 2024

@pnicolli ho verificato e sistemato l'accessibilità.
C'erano in generale dei problemi con la navigazione da tastiera e lo screen-reader.
Ho creato dei componenti generici per gli slider che sistemano l'accessibilità e li ho usati ovunque vengono fatti degli slider.
In questo modo abbiamo tutto coerente.

Dimenticavo: gli interventi di accessibilità che ho fatto sullo slider, li ho fatti guardando come è stato fatto https://accessible360.github.io/accessible-slick/.

@pnicolli pnicolli added the rer label Mar 5, 2024
@giuliaghisini
Copy link
Author

@pnicolli dovrei avere sistemato tutto, anche i titoli con lo SR nel caso di slider con sole immagini

@pnicolli pnicolli merged commit a37e985 into main Apr 8, 2024
3 checks passed
@pnicolli pnicolli deleted the appearance_card_in_slider_template branch April 8, 2024 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants