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(styles): update Dialog to latest Horizon [ci visual] #4536

Merged
merged 1 commit into from May 24, 2023

Conversation

InnaAtanasova
Copy link
Contributor

Related Issue

Closes #4375

Description

  • updates the resize handle
  • a bit of code refactoring
  • added an example for device specifications regarding the resize handle and full screen button

@InnaAtanasova InnaAtanasova added the Horizon 2023 Horizon Design Changes in 2023 label May 17, 2023
@InnaAtanasova InnaAtanasova added this to the Sprint 113 milestone May 17, 2023
@InnaAtanasova InnaAtanasova requested a review from a team May 17, 2023 16:47
@InnaAtanasova InnaAtanasova self-assigned this May 17, 2023
@netlify
Copy link

netlify bot commented May 17, 2023

Deploy Preview for fundamental-styles ready!

Name Link
🔨 Latest commit 0a2a507
🔍 Latest deploy log https://app.netlify.com/sites/fundamental-styles/deploys/646e3dc1e8793e000857e816
😎 Deploy Preview https://deploy-preview-4536--fundamental-styles.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.

Copy link
Contributor

@g-cheishvili g-cheishvili left a comment

Choose a reason for hiding this comment

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

Only thing that I have is the naming of the story, which is optional, because there are plenty of other cases where we do that

Comment on lines 201 to 202
export const DeviceSpecifications = () => deviceSpecificationsExampleHtml;
DeviceSpecifications.storyName = 'Tablet and Hybrid Device Specification';
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you name the story const same as story name and remove DeviceSpecifications.storyName assignment?

@droshev
Copy link
Contributor

droshev commented May 24, 2023

the RTL expander looks off
Screenshot 2023-05-23 at 9 12 31 PM

@InnaAtanasova
Copy link
Contributor Author

the RTL expander looks off Screenshot 2023-05-23 at 9 12 31 PM

fixed the resize handler in RTL

@droshev droshev merged commit c07796e into main May 24, 2023
13 checks passed
@droshev droshev deleted the fix/dialog-horizon-2023 branch May 24, 2023 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Horizon 2023 Horizon Design Changes in 2023
Projects
Development
  
Awaiting triage
Development

Successfully merging this pull request may close these issues.

Component Design Update: Dialog
3 participants