feat:navbar stylization #64
Conversation
Konzum59
force-pushed
the
feat/navbar-stylization
branch
from
c6f9a14 to
7942f8c
Compare
src/components/navbar.tsx
Outdated
| <div className="flex gap-[25px] text-lg font-semibold lowercase leading-6"> | ||
| kup bilet | ||
| <div className="flex h-[25px] w-[25px] shrink-0 items-center justify-center"> | ||
| <svg |
There was a problem hiding this comment.
Troche nie rozumiem konceptu uzywania svg w ten sposob w kodzie, w przypadku tego przycisku kup bilet moze lepiej uzyc stylizowanego diva? Pamiętaj też że docelowo kup bilet to link stylizowany na przycisk
src/components/navbar.tsx
Outdated
| </div> | ||
| </div> | ||
| </div> | ||
| <svg |
There was a problem hiding this comment.
Tutaj tez to svg, nie wydaje mi sie zeby to byl dobry koncept zeby sam wektor byl przyciskiem, raczej uzyj taga button i wewnatrz umieszczaj ikonki. Obecne rozwiazanie psuje dostepnosc i nie jest to do konca poprawne semantycznie.
|
I staraj się też używać tych tailwindowych wartości nie sztywnych w px, tam gdzie sie da |
Octopus4344
force-pushed
the
feat/navbar-stylization
branch
from
7942f8c to
3aa7c29
Compare
Konzum59
force-pushed
the
feat/navbar-stylization
branch
from
3aa7c29 to
12f8e0c
Compare
There was a problem hiding this comment.
Wieksze odleglosci miedzy linkami dla sm i md
src/components/navbar-mobile.tsx
Outdated
| </div> | ||
| </div> | ||
| </div> | ||
| <SheetDescription></SheetDescription> |
There was a problem hiding this comment.
To mozna usunac, po co pusty tag
src/components/navbar-mobile.tsx
Outdated
| Aktualności | ||
| </Link> | ||
| </div> | ||
| <div className="absolute bottom-20 flex rounded-[30px] bg-gradient-main px-8 py-5 text-white"> |
There was a problem hiding this comment.
Czemu nie uzyjesz zdefiniowanej klast dla border radiusa? Nie lubie uzywania tych sztywnych wartosci
There was a problem hiding this comment.
Mniejszy font, taki zeby nic nie lamalo sie na dwie linijki w standardowej moblice
src/components/navbar.tsx
Outdated
| <Link | ||
| href="/artists" | ||
| aria-label="Przejdź do strony z artystami" | ||
| className="rounded-full hover:bg-gradient-main sm:p-1 lg:p-2" |
There was a problem hiding this comment.
Proponuje dodac jakis maly komponent do tego albo customowa klase tailwindowa, bo to sa powtarzalne elementy
src/components/navbar-mobile.tsx
Outdated
| /> | ||
| </SheetHeader> | ||
| <div className="flex flex-1 flex-col items-center justify-end gap-2.5 px-3 py-7"> | ||
| <div className="w-62 flex h-20 shrink-0 items-center justify-between"></div> |
There was a problem hiding this comment.
czemu tu jest pusty div? Jeśli jest tylko przerwa to lepiej zwiększyć odległości po prostu. Trochę też to nestowanie wielu divów nieładne, zobacz czy nie możesz tego jakoś elegancko
| </div> | ||
| <div className="absolute bottom-2 right-4"> | ||
| <NavbarMobile onButtonClick={changeLogoState} /> | ||
| </div> |
There was a problem hiding this comment.
W sumie to może być hidden nie w moblice
Octopus4344
force-pushed
the
feat/navbar-stylization
branch
from
747dae4 to
ed45324
Compare
Octopus4344
force-pushed
the
feat/navbar-stylization
branch
from
529cb77 to
35d05d3
Compare
No description provided.