fix: position of badges#4657
Conversation
|
@Udit-takkar is attempting to deploy a commit to the cal Team on Vercel. A member of the Team first needs to authorize it. |
|
|
hariombalhara
added
the
🎨 needs design
|
We should definitely show the label on mobile. As well as the date & time. We have designs for this here.
It would be great to also test this with recurring bookings. |
|
ok |
|
should i also change the button(Edit booking) like you have done here ? |
|
build this for mobile view |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@Udit-takkar left a comment re: not using |
Head branch was pushed to by a user without write access
|
done |
PeerRich
left a comment
PeerRich
left a comment
There was a problem hiding this comment.
great stuff! thank you @Udit-takkar
PeerRich
removed
the
🎨 needs design
hariombalhara
added
♻️ autoupdate
|
Just a FYI @Udit-takkar @PeerRich this PR is what currently breaking our E2E tests. We're introducing duplicate markdown and displaying it with breakpoints and Playwright doesn't like that. |
| <div className="flex justify-end sm:hidden"> | ||
| <div className="whitespace-nowrap pt-2 pb-4 text-right text-sm font-medium ltr:pr-4 rtl:pl-4"> | ||
| {isUpcoming && !isCancelled ? ( | ||
| <> | ||
| {isPending && user?.id === booking.user?.id && <TableActions actions={pendingActions} />} | ||
| {isConfirmed && <TableActions actions={bookedActions} />} | ||
| {isRejected && <div className="text-sm text-gray-500">{t("rejected")}</div>} | ||
| </> | ||
| ) : null} | ||
| {isPast && isPending && !isConfirmed ? <TableActions actions={bookedActions} /> : null} | ||
| </div> | ||
| </div> |
There was a problem hiding this comment.
This code block specifically is what is breaking our tests:
5 participants
fixes: #4649