Navbar - Githubissues

roertbb commented 2 years ago

What was one in PR:

navigation (desktop and mobile) for onepage
move akai logo to navbar
adjust icons order
adjust section naming

nav_desktop

nav_mobile

tobiaszciesielski commented 2 years ago

Masz approve ode mnie ale przejrzyj to co napisałem. Gj!

roertbb commented 2 years ago

Po kilku rundach testowania widzę, że higlightowanie elementów nie działa jeszcze do końca tak, jak powinno - w wolnej chwili poczytam jeszcze trochę o Intersection Observer API i postaram się podejść bardziej merytorycznie do tematu :wink:

roertbb commented 2 years ago

Naprawiłem wykrywanie sekcji i po zmianach z https://github.com/akai-org/akai-webpage/pull/45/commits/3b4b1229a503028a774761f2c6e9a3e992926c80 powinno to poprawnie działać, niestety nie udało mi się tego obejść bez obsługiwania edge-case'ów, w związku z nieregularnością poszczególnych sekcji 😞

Niżej podrzucam gifa, jak to wygląda przybliżone zachowanie Intersection Observer - widać, że jeśli zjedziemy do ostatniej sekcji, to nie dostaniemy eventu, że wjeżdżamy ponownie na przed ostatnią przy scrollowaniu do góry - stąd ten edge-case jest wyłapany i obsłużony oddzielnie. Pytanie - czy można spełnić nasze wymagania bez tego dodatkowego if'a 🤔

ezgif-2-f7a51702f2

akai-org / akai-webpage

Navbar #45