search

fdnd-agency / buurtcampus-oost

In Buurtcampus Oost zoeken studenten samen met bewoners, professionals, ondernemers, docenten en onderzoekers gezamenlijk naar oplossingen voor kansrijke uitdagingen uit de buurt.
MIT License
1 stars 11 forks source link

Hero slider - feedback #62

Closed JalalToufik closed 6 months ago

JalalToufik commented 6 months ago

https://github.com/fdnd-agency/buurtcampus-oost/issues/47

ju5tu5 2 days ago Mooi component! Het zou helemaal gaaf zijn als je de drie assets meestuurt vanuit de plaats waar de hero-slider wordt aangeroepen. Daarmee maak je het molecule universeel toepasbaar, als je in een volgend project een keer een slider nodig hebt pak je dit component er gewoon bij.. dat is nog eens DRY!

krijnhoetmer commented 1 hour ago Mooi dingetje! Net al even met je gezeten, maar een paar puntjes:

Responsive images gebruiken voor je header (dus met ). Sowieso voor verschillende image types (jpeg, webp, avif), maar misschien ook wel verschillende formaten. Lazy loading even goed uitzoeken; staat nu verkeerd in de code width/height attributen op je images zetten Gebruik van aria-label kan beter. Voegen deze images met deze labels écht wat toe in screen readers?

JalalToufik commented 6 months ago

Ik heb de feedback verwerkt en gepusht !