Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(date-picker): sync code and figma #1899

Merged
merged 5 commits into from
May 31, 2023
Merged

fix(date-picker): sync code and figma #1899

merged 5 commits into from
May 31, 2023

Conversation

nowseemee
Copy link
Collaborator

@nowseemee nowseemee commented May 30, 2023

@felix-ico could you take a look at this, please?

@netlify
Copy link

netlify bot commented May 30, 2023

Deploy Preview for marvelous-moxie-a6e2fe ready!

Name Link
🔨 Latest commit cc74814
🔍 Latest deploy log https://app.netlify.com/sites/marvelous-moxie-a6e2fe/deploys/6477100bf52122000899cb32
😎 Deploy Preview https://deploy-preview-1899--marvelous-moxie-a6e2fe.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@nowseemee nowseemee force-pushed the datepicker-styles branch 2 times, most recently from 0c1cb38 to 8754d75 Compare May 30, 2023 12:16
@nowseemee nowseemee force-pushed the datepicker-styles branch from 8754d75 to 1e9637f Compare May 30, 2023 12:37
@nowseemee nowseemee marked this pull request as ready for review May 30, 2023 12:59
@nowseemee nowseemee requested a review from acstll as a code owner May 30, 2023 12:59
@nowseemee nowseemee requested review from felix-ico and removed request for acstll May 30, 2023 12:59
@felix-ico
Copy link
Collaborator

felix-ico commented May 30, 2023

@nowseemee
Screenshot 2023-05-30 at 16 46 24
Screenshot 2023-05-30 at 16 47 14

i think the top-padding here should be 16px (and maybe the bottom padding/margin needs to be adjusted)

@felix-ico
Copy link
Collaborator

Screenshot 2023-05-30 at 16 49 57

Also not sure if this is expected but there is a close icon that can be tabbed to (otherwise it is invisible) and previously it was on the top right, now on the top left

@nowseemee nowseemee force-pushed the datepicker-styles branch from e67dcc7 to 1c757f8 Compare May 31, 2023 07:20
@nowseemee nowseemee force-pushed the datepicker-styles branch from b95b8d6 to ec5549b Compare May 31, 2023 08:56
@nowseemee nowseemee merged commit 305a42a into main May 31, 2023
@nowseemee nowseemee deleted the datepicker-styles branch May 31, 2023 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants