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

UploadField component adapted to the new design system #545

Merged
merged 8 commits into from
Dec 2, 2022

Conversation

vmilan
Copy link
Contributor

@vmilan vmilan commented Dec 2, 2022

Description

Shortcut: #268673
[sc-268673]

  • UploadField component with the new design system applied.
  • Removed InputFile (is replaced by UploadField)
  • Also added some minor details in Select (design review).

@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #268673: COMPONENTS - Basic components (forms).

@vmilan vmilan marked this pull request as ready for review December 2, 2022 16:51
@vmilan vmilan requested a review from a team December 2, 2022 16:51
placeholder: 'Drag and drop your file or click to browse',
dragPlaceholder: 'Drop your file here...',
buttonText: 'Browse',
accept: 'application/JSON',
Copy link
Contributor

@VictorVelarde VictorVelarde Dec 2, 2022

Choose a reason for hiding this comment

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

Is this comma separated, in the case of various types? Should we use the opportunity to have accept as an array externally?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I updated the prop, I think I got you right, if not feel free to modify anything. Thank you!

@@ -216,7 +213,7 @@ type TooltipDataProps = {
};

// SelectField
export interface SelectFieldProps {
export interface SelectFieldProps extends TextFieldProps {
Copy link
Contributor

Choose a reason for hiding this comment

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

👍🏻

@VictorVelarde VictorVelarde merged commit 7753293 into feature/design-system Dec 2, 2022
@VictorVelarde VictorVelarde deleted the feature/upload-field branch December 2, 2022 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants