62 konverter til tailwindcss #102
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Noen som kan sjekke hvorfor Vercel ikke deployer? |
|
[16:33:41.057] Running build in Cleveland, USA (East) – cle1 |
|
Du kan kjøre npm run build før du pusher, er det som kjøres på Vercel 😊 |
app/layout.tsx
Outdated
There was a problem hiding this comment.
Her tror jeg noe må justeres, fordi det virker som at nå bruker en side mye mindre plass på siden fordi det er mye mer margin på høyre og venstre side. Se her for eksempel:
vs
There was a problem hiding this comment.
Dette er nok pga max-width på 960px på layout komponenten. Utivklet for det meste på min skjerm så har ikke hatt tilgang på høyere breakpoints som sikkert definerer en større max-width... Du har ikke tilang på storskjerm og kan se hva den er på her?
There was a problem hiding this comment.
vettuhva, så over litt til og det er ut som det er bare partner siden som er rar her og resten av sidene har en max-width på 960 uavhengig av størrelsen på skjermen. Forksjellen nå er at partner siden følger også den regelen på i min branch.. Vi kan jo ta en runde på om dette skal justeres ved større breakpoint tho.
| <AnimatedDivider /> | ||
| <AnimatedFireworks /> | ||
| </div> | ||
|
|
||
| <h1 className={`title ${styles.fontAndColor} mt-4`}> | ||
| <Title className={`mt-4 font-['Noto Serif', 'Open Sans', 'serif'] `}> |
There was a problem hiding this comment.
Jeg tenker at det er lurt om vi kan definere fontene ett sted i koden, slik at det er lettere å følge og mindre rom for feil
There was a problem hiding this comment.
ref. kommentaren min på "subtitle"
components/section/Section.tsx
Outdated
|
|
||
| interface SectionProps { | ||
| /** | ||
| * TODO |
| return ( | ||
| <a | ||
| className={`text-big-text-color hover:text-hover-text-color ${className}`} | ||
| href={href} |
There was a problem hiding this comment.
Legge til rel={"noreferrer noopener""}?
| ) | ||
| case 'subtitle': | ||
| return ( | ||
| <h1 |
There was a problem hiding this comment.
Subtitle bør vel ikke være h1 mtp accessibility? Tenker det gir mer mening med enten h2-h6 eller at man kan spesifisere nivået
There was a problem hiding this comment.
Dette er en av tingene jeg tenkte vi kunne ta i en senere iterasjon. Det var litt forskjellig bruk av størrelser på sidene jeg gikk igjennom og tenkte vi kunne tatt en runde på det sammen.
| /** | ||
| * Primary UI component for user interaction | ||
| */ | ||
| export const Title = ({ type = 'title', className, children, ...props }: TitleProps) => { |
There was a problem hiding this comment.
Kanskje man bør kunne si hvilken level man vil ha, f.eks 1-6 også rendrer vi h1-h6 alt avhengig? Mulig med default på h1
There was a problem hiding this comment.
Ref, kommentaren min over på "subtitle"
daudmohamed
force-pushed
the
62-konverter-til-tailwindcss
branch
from
5bc272f
to
457ee3d
Compare
Det tok litt tid, men nå har jeg fjernet bulma helt og lagt til tailwind. Jeg har også lagt til storybook og tatt det litt i bruk, men tenker det er fint å ha for kjapp prototyping i fremtiden. Jeg har for det meste fokusert på å beholde den samme looken som med bulma, men utenom noen små forskjeller på tekststørrelser så syns jeg det ser greit ut egentlig.
Lykke til med å gå igjennom :)