Skip to content

fix: Push mini calendar down in week / column view#9628

Merged
PeerRich merged 1 commit into
mainfrom
fix/remove-height-100percent
Jun 19, 2023
Merged

fix: Push mini calendar down in week / column view#9628
PeerRich merged 1 commit into
mainfrom
fix/remove-height-100percent

Conversation

@JeroenReumkens
Copy link
Copy Markdown
Contributor

What does this PR do?

Removed full height which incorrectly limited height in booker sidebar. Tested in Chrome, FF and Safari.

Before

image

After

CleanShot 2023-06-19 at 14 55 14@2x

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

  • Ensure mini calendar in weekview / column view is pushed down.
  • Ensure calendar looks okay on mobile too

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 19, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 19, 2023 1:09pm
cal ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 19, 2023 1:09pm
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 19, 2023 1:09pm
web-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 19, 2023 1:09pm

@JeroenReumkens JeroenReumkens changed the title Removed full height which incorrectly limited height in booker sideba… fix: Push mini calendar down in week / column view Jun 19, 2023
)}>
<BookerSection
area="meta"
className="max-w-screen flex h-full w-full flex-col md:w-[var(--booker-meta-width)]">
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This incorrectly limited the meta side column height if contents was smaller than screen.

@github-actions
Copy link
Copy Markdown
Contributor

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@deploysentinel
Copy link
Copy Markdown

deploysentinel Bot commented Jun 19, 2023

Current Playwright Test Results Summary

✅ 98 Passing - ⚠️ 3 Flaky

Run may still be in progress, this comment will be updated as current testing workflow or job completes...

(Last updated on 06/19/2023 01:13:41pm UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: 7bd659c

Started: 06/19/2023 01:09:23pm UTC

⚠️ Flakes

📄   apps/web/playwright/login.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
user can login & logout succesfully login flow user & logout using dashboard
Retry 1Initial Attempt
4.48% (13) 13 / 290 runs
failed over last 7 days
37.93% (110) 110 / 290 runs
flaked over last 7 days

📄   apps/web/playwright/event-types.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Event Types tests user -- old-booker can add multiple organizer address
Retry 1Initial Attempt
1.42% (4) 4 / 281 runs
failed over last 7 days
28.47% (80) 80 / 281 runs
flaked over last 7 days

📄   apps/web/playwright/webhook.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
FORM_SUBMITTED can submit a form and get a submission event
Retry 1Initial Attempt
4.35% (9) 9 / 207 runs
failed over last 7 days
15.94% (33) 33 / 207 runs
flaked over last 7 days

View Detailed Build Results


@PeerRich PeerRich merged commit 84ad133 into main Jun 19, 2023
@PeerRich PeerRich deleted the fix/remove-height-100percent branch June 19, 2023 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants