Skip to content

chore: small changes to dropdown & main sidebar#9719

Merged
PeerRich merged 12 commits into
mainfrom
ui/minor-bugs
Jul 2, 2023
Merged

chore: small changes to dropdown & main sidebar#9719
PeerRich merged 12 commits into
mainfrom
ui/minor-bugs

Conversation

@ciaranha
Copy link
Copy Markdown
Member

What does this PR do?

Small changes to filter dropdown. Improved border colors, font-weights, focus state, icons size, spacing around dividers. Other small changes to sidebar. Making items a little more compact.

CleanShot.2023-06-22.at.16.01.31.mp4

Type of change

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

How should this be tested?

  • Go to Routing forms and check out the filter dropdown.
  • Test the User dropdown in the main sidebar

@ciaranha ciaranha requested a review from sean-brydon June 22, 2023 15:03
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 22, 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 Mar 19, 2024 3:18pm
cal ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 19, 2024 3:18pm
ui ❌ Failed (Inspect) Mar 19, 2024 3:18pm
2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
dev ⬜️ Ignored (Inspect) Mar 19, 2024 3:18pm
web-staging ⬜️ Ignored (Inspect) Visit Preview Mar 19, 2024 3:18pm

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 22, 2023

Thank you for following the naming conventions! 🙏

@ciaranha ciaranha added the 🧹 Improvements Improvements to existing features. Mostly UX/UI label Jun 22, 2023
Copy link
Copy Markdown
Member

@sean-brydon sean-brydon left a comment

Choose a reason for hiding this comment

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

LGTM - Glad us setting up your env didnt take hours <3

)}>
<Avatar
size={small ? "xs" : "sm"}
size={small ? "xs" : "xsm"}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Was this in intentional

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yea I want the avatar to be 20x20px - so if that's indeed what this does - then yes!

Copy link
Copy Markdown
Member

@sean-brydon sean-brydon left a comment

Choose a reason for hiding this comment

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

Requesting changes while we figure something out

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 22, 2023

📦 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! 🙌

@PeerRich PeerRich changed the title small changes to dropdown & main sidebar chore: small changes to dropdown & main sidebar Jun 22, 2023
@PeerRich
Copy link
Copy Markdown
Member

Requesting changes while we figure something out

figured out? cause i tested everytrhing and it looks fine

PeerRich
PeerRich previously approved these changes Jun 22, 2023
@PeerRich PeerRich enabled auto-merge (squash) June 22, 2023 19:41
@sean-brydon
Copy link
Copy Markdown
Member

Requesting changes while we figure something out

figured out? cause i tested everytrhing and it looks fine

Yeah all good - We spotted something with nav items but i dont think its coming from this PR - if it is i'll follow up tomorrow

@sean-brydon sean-brydon self-requested a review June 22, 2023 19:41
sean-brydon
sean-brydon previously approved these changes Jun 22, 2023
Comment thread packages/ui/components/empty-screen/EmptyScreen.tsx Outdated
Comment thread packages/ui/components/popover/AnimatedPopover.tsx Outdated
PeerRich
PeerRich previously approved these changes Jul 1, 2023
Comment thread packages/ui/components/empty-screen/EmptyScreen.tsx Outdated
PeerRich
PeerRich previously approved these changes Jul 2, 2023
Comment thread packages/ui/components/empty-screen/EmptyScreen.tsx Outdated
@deploysentinel
Copy link
Copy Markdown

deploysentinel Bot commented Jul 2, 2023

Current Playwright Test Results Summary

✅ 101 Passing - ⚠️ 2 Flaky

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

(Last updated on 07/02/2023 10:12:14am UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: 4537122

Started: 07/02/2023 10:10:39am UTC

⚠️ Flakes

📄   apps/web/playwright/booking-pages.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
pro user -- new-booker can book an event that requires confirmation and then that booking can be accepted by organizer
Retry 1Initial Attempt
0.49% (1) 1 / 205 run
failed over last 7 days
3.90% (8) 8 / 205 runs
flaked over last 7 days

📄   apps/web/playwright/booking-seats.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Reschedule for booking with seats -- old-booker Should book with seats and hide attendees info from showAttendees true
Retry 1Initial Attempt
0% (0) 0 / 206 runs
failed over last 7 days
0.97% (2) 2 / 206 runs
flaked over last 7 days

View Detailed Build Results


@PeerRich PeerRich merged commit 6fb51db into main Jul 2, 2023
@PeerRich PeerRich deleted the ui/minor-bugs branch July 2, 2023 10:12
@PeerRich PeerRich added the core area: core, team members only label Jul 24, 2023
fritterhoff pushed a commit to hm-edu/cal.com that referenced this pull request Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only 🧹 Improvements Improvements to existing features. Mostly UX/UI Low priority Created by Linear-GitHub Sync ui area: UI, frontend, button, form, input

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants