Skip to content

fix: responsive upload avatar#10392

Merged
emrysal merged 2 commits into
mainfrom
fix/upload-avatar
Jul 26, 2023
Merged

fix: responsive upload avatar#10392
emrysal merged 2 commits into
mainfrom
fix/upload-avatar

Conversation

@Udit-takkar
Copy link
Copy Markdown
Contributor

@Udit-takkar Udit-takkar commented Jul 26, 2023

What does this PR do?

Fixes #10385

Screenshot 2023-07-26 at 4 10 07 PM

Requirement/Documentation

  • If there is a requirement document, please, share it here.
  • If there is ab UI/UX design document, please, share it here.

Type of change

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

Signed-off-by: Udit Takkar <udit.07814802719@cse.mait.ac.in>
@vercel
Copy link
Copy Markdown

vercel Bot commented Jul 26, 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 Jul 26, 2023 11:45am
cal-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 26, 2023 11:45am
dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 26, 2023 11:45am
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 26, 2023 11:45am
2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Jul 26, 2023 11:45am
qa ⬜️ Ignored (Inspect) Jul 26, 2023 11:45am

@github-actions github-actions Bot added 🎨 needs design Before engineering kick-off, a designer needs to submit a mockup 👩‍🔬 needs investigation 🧹 Improvements Improvements to existing features. Mostly UX/UI labels Jul 26, 2023
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 26, 2023

Thank you for following the naming conventions! 🙏

@alwaysmeticulous
Copy link
Copy Markdown

alwaysmeticulous Bot commented Jul 26, 2023

🤖 Meticulous spotted visual differences in 65 of 304 screens tested: view and approve differences detected.

Last updated for commit e18b120. This comment will update as new commits are pushed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 26, 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! 🙌

@deploysentinel
Copy link
Copy Markdown

deploysentinel Bot commented Jul 26, 2023

Current Playwright Test Results Summary

✅ 105 Passing - ⚠️ 1 Flaky

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

(Last updated on 07/26/2023 11:51:59am UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: e18b120

Started: 07/26/2023 11:47:34am UTC

⚠️ Flakes

📄   packages/embeds/embed-core/playwright/tests/action-based.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Popup Tests should be able to reschedule
Retry 1Initial Attempt
1.52% (3) 3 / 197 runs
failed over last 7 days
97.97% (193) 193 / 197 runs
flaked over last 7 days

View Detailed Build Results


Move title to the DialogContent title prop instead of the content.
Copy link
Copy Markdown
Contributor

@emrysal emrysal left a comment

Choose a reason for hiding this comment

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

Good to go, thanks @Udit-takkar

@emrysal emrysal enabled auto-merge (squash) July 26, 2023 11:46
@emrysal emrysal merged commit 69c01b6 into main Jul 26, 2023
@emrysal emrysal deleted the fix/upload-avatar branch July 26, 2023 11:52
joeauyeung pushed a commit that referenced this pull request Jul 26, 2023
* fix: responsive upload avatar

Signed-off-by: Udit Takkar <udit.07814802719@cse.mait.ac.in>

* Update ImageUploader.tsx

Move title to the DialogContent title prop instead of the content.

---------

Signed-off-by: Udit Takkar <udit.07814802719@cse.mait.ac.in>
Co-authored-by: Alex van Andel <me@alexvanandel.com>
sean-brydon pushed a commit that referenced this pull request Jul 31, 2023
* fix: responsive upload avatar

Signed-off-by: Udit Takkar <udit.07814802719@cse.mait.ac.in>

* Update ImageUploader.tsx

Move title to the DialogContent title prop instead of the content.

---------

Signed-off-by: Udit Takkar <udit.07814802719@cse.mait.ac.in>
Co-authored-by: Alex van Andel <me@alexvanandel.com>
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 🎨 needs design Before engineering kick-off, a designer needs to submit a mockup

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

Upload Avtar should have some padding at the bottom

3 participants