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

Featured presentations on frontpage #40

Closed
Tschonti opened this issue Feb 9, 2024 · 0 comments · Fixed by #44
Closed

Featured presentations on frontpage #40

Tschonti opened this issue Feb 9, 2024 · 0 comments · Fixed by #44
Assignees

Comments

@Tschonti
Copy link
Member

Tschonti commented Feb 9, 2024

Két előadást a főoldalon is szeretnénk kiemelni.
https://www.figma.com/file/ATBcUtDFuUxF55PHYUPD9s/konf-web?type=design&node-id=786-2944&mode=design&t=WxVik0FwCRrjKMSz-4
(a figmán csak az első van rajta, a másodiknak a promó videó után kéne lennie, pont ugyanúgy mint az elsőnek)
A megjelenésük nagyon hasonlít az előadás részletes oldalához, de van két jelentős különbség: a főoldalon egy csempében van az egész, illetve a cím elrendezése is más. Szebb lenne, ha ugyanazt a komponenst használná mindkét esetben az oldal (elég ha csak a kontent van ebben a komponensben, és akkor a Tile komponens körül tudja venni a főoldalon. A cím elrendezését meg valahogy propokkal meg lehet oldalni). Ha ez túl sok effortnak tűnik, lehet akár különböző a kód...

A főoldalon két előadás adatai majd nem az apiról fognak jönni, hanem hardcodeolva lesznek, de egyelőre bőven jó ha az apiról érkező első két előadást jelenítjük meg.
Az extra logók (pl Kotlin a figmán) csak a hardcodeolt előadásoknál lesznek, ezért érdemes a Presenation modelt kibővíteni mondjuk egy optional imageUrls tömbbel

A figmán nincs rajt, de szerintem érdemes lenne tenni két gombot a csempébe: egyik az adott előadás részletes oldalára vezet, másik pedig az összes előadás listájára. Erre rákérdezek még discordon

@Tschonti Tschonti linked a pull request Feb 13, 2024 that will close this issue
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 a pull request may close this issue.

2 participants