feat: profile component #282
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,54 @@ | ||||||||||||||||||
| @props([ | ||||||||||||||||||
| 'user', | ||||||||||||||||||
| 'buttons', | ||||||||||||||||||
| 'stats' => [ | ||||||||||||||||||
| 'lists' => 0, | ||||||||||||||||||
| 'reviews' => 0, | ||||||||||||||||||
| 'friends' => 0, | ||||||||||||||||||
| ], | ||||||||||||||||||
| ]) | ||||||||||||||||||
|
|
||||||||||||||||||
| <div class="relative flex w-full"> | ||||||||||||||||||
| <div class="absolute top-0 right-0"> | ||||||||||||||||||
| <x-lucide-ellipsis-vertical class="h-6 w-6 text-slate-50" /> | ||||||||||||||||||
dcbwe marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||||
| </div> | ||||||||||||||||||
|
|
||||||||||||||||||
| <div class="mr-6 flex w-full flex-row flex-wrap items-center gap-4"> | ||||||||||||||||||
ell-ska marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||||
| <div class="flex size-30 flex-none items-center justify-center"> | ||||||||||||||||||
| @isset($user) | ||||||||||||||||||
| <x-avatar :image="$user->image" size="lg" /> | ||||||||||||||||||
| @else | ||||||||||||||||||
| <x-avatar size="lg" /> | ||||||||||||||||||
| @endisset | ||||||||||||||||||
| </div> | ||||||||||||||||||
|
There was a problem hiding this comment. i don't think there needs to be a wrapping div for the avatar. there's also no need to check if there is a
Suggested change
There was a problem hiding this comment. Är wrapper onödig tar vi bort den. Check finns för att inte krasha i test There was a problem hiding this comment. vilka test? denna komponent används bara på profilsidan och man kan inte gå in på en profil som inte existerar. då kommer man komma till 404 innan denna sida renderar |
||||||||||||||||||
|
|
||||||||||||||||||
| <div class="flex grow flex-col gap-3 pr-10 text-slate-50"> | ||||||||||||||||||
|
There was a problem hiding this comment. följer figmas design, kan tas bort There was a problem hiding this comment. finns ingen padding på denna div i figma? |
||||||||||||||||||
| <div class="text-lg font-bold"> | ||||||||||||||||||
| @isset($user) | ||||||||||||||||||
| {{ $user->username }} | ||||||||||||||||||
| @else | ||||||||||||||||||
| @username | ||||||||||||||||||
| @endisset | ||||||||||||||||||
|
There was a problem hiding this comment.
Suggested change
same reason as above |
||||||||||||||||||
| </div> | ||||||||||||||||||
| <div class="flex flex-row gap-6 text-slate-50"> | ||||||||||||||||||
dcbwe marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||||
| @foreach ($stats as $stat => $sum) | ||||||||||||||||||
| <div class="flex flex-col"> | ||||||||||||||||||
| <p class="text-sm font-bold"> | ||||||||||||||||||
| {{ $sum }} | ||||||||||||||||||
| </p> | ||||||||||||||||||
| <span class="text-sm font-normal"> | ||||||||||||||||||
|
There was a problem hiding this comment.
Suggested change
|
||||||||||||||||||
| {{ $stat }} | ||||||||||||||||||
| </span> | ||||||||||||||||||
| </div> | ||||||||||||||||||
| @endforeach | ||||||||||||||||||
| </div> | ||||||||||||||||||
| </div> | ||||||||||||||||||
|
|
||||||||||||||||||
| <div class="flex flex-none flex-row gap-2 self-start"> | ||||||||||||||||||
ell-ska marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||||
| @isset($buttons) | ||||||||||||||||||
| {{ $buttons }} | ||||||||||||||||||
| @endisset | ||||||||||||||||||
| </div> | ||||||||||||||||||
| </div> | ||||||||||||||||||
| </div> | ||||||||||||||||||
ell-ska marked this conversation as resolved.
Show resolved
Hide resolved
There was a problem hiding this comment. again, please don't remove the layout There was a problem hiding this comment. du har raderat allt innehåll i denna fil. det ska vara där There was a problem hiding this comment. den här filen finns det nu innehåll i i main. jag vet inte varför du inte får en merge konflikt men se till att du inte tar bort det som ligger i main innan denna mergas |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i don't think you can pass html as a prop? and that this rather should be a named slot. but correct me if i'm wrong!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
absolut inte säker?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
om du har testat och det funkar så är det lugnt men vill veta att det funkar innan vi mergar detta