Feature/Embed/Add Embed ui instruction config to hide event type description#5951
Conversation
CAL-529 Hide event type details in embed
Hi! We are looking for a way to embed the Cal.Com's calendar to our website without event type details. Expected behaviour would be embedding the calendar widget only, without event name, host's name, location or duration. This is important for us for a smoother experience on mobile - 95% of our traffic is mobile :) We used to use the Hide event details option in Calendly - I'll attach a screenshot, might be useful m Tagging @alishaz-polymath and @hariom as we discussed this on a call today :) |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
hariombalhara
changed the title
ui instruction config to hide event type description
hariombalhara
force-pushed
the
5792-cal-529-hide-event-type-details-in-embed
branch
from
c63a46b to
4b3516d
Compare
hariombalhara
force-pushed
the
5792-cal-529-hide-event-type-details-in-embed
branch
from
4b3516d to
daff63f
Compare
|
@Jaibles FYI, This is a new toggle in Embed. Please check the loom. Being a very small change, I think designs are not required. |
hariombalhara
left a comment
hariombalhara
left a comment
There was a problem hiding this comment.
Self Review done
| {embed.title} | ||
| </h3> | ||
| <hr className={classNames("mt-4", embedType === "element-click" ? "hidden" : "")} /> | ||
| <div className="max-h-97 flex flex-col overflow-y-auto"> |
There was a problem hiding this comment.
BugFix: On big screens, this was unnecessarily restricting the view like this
| <div className="text-sm">Embed Window Sizing</div> | ||
| <div className="justify-left flex items-center"> | ||
| <TextField | ||
| name="width" |
There was a problem hiding this comment.
BugFix: This was adding the labels with values the same as name unnecessarily.
| theme: Theme.auto, | ||
| floatingPopup: {}, | ||
| elementClick: {}, | ||
| hideEventTypeDetails: false, |
There was a problem hiding this comment.
By default keep it disabled.
| <div> | ||
| <main | ||
| className={classNames( | ||
| "flex-col md:mx-4 lg:flex", |
There was a problem hiding this comment.
| </div> --> | ||
| </div> | ||
| <h2>Inline Embed Examples</h2> | ||
| <div id="namespaces-test"> |
There was a problem hiding this comment.
Following is some cleanup on Embed Playground page to keep it clean.
| ); | ||
| } | ||
|
|
||
| if (only === "all" || only === "hideEventTypeDetails") { |
There was a problem hiding this comment.
Add hideEventTypeDetails demo in playground
…nt-type-details-in-embed
hariombalhara
force-pushed
the
5792-cal-529-hide-event-type-details-in-embed
branch
from
daff63f to
2131ebf
Compare
…nt-type-details-in-embed
hariombalhara
added
♻️ autoupdate
kodiakhq
Bot
deleted the
5792-cal-529-hide-event-type-details-in-embed
branch
4 participants
What does this PR do?
Fixes #5792
Demo Loom
Environment: Staging(main branch) / Production
Type of change
How should this be tested?
Checklist