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

PresentationStyle #311

Merged
merged 7 commits into from
Aug 28, 2023
Merged

PresentationStyle #311

merged 7 commits into from
Aug 28, 2023

Conversation

SharfMohiuddin
Copy link
Contributor

pageSheet

Added the prop for presentationStyle also removed the warning of transparent if presentationStyle === 'pageSheet'
@RichardLindhout
Copy link
Member

Looks good! Can you update it with the right prettier? Seems like some things are different (; is added)

Used the right prettier and removed the ; from the code.
@SharfMohiuddin
Copy link
Contributor Author

I just used the right prettier in this commit. Kindly, accept the pull request. @RichardLindhout

@iM-GeeKy
Copy link
Collaborator

iM-GeeKy commented Aug 24, 2023

@RichardLindhout and @SharfMohiuddin I think we should also add this new prop to the documentation (Date Picker), so it is visible to others who might want to use this in the future.

@SharfMohiuddin
Copy link
Contributor Author

I agree with you. @iM-GeeKy

@SharfMohiuddin
Copy link
Contributor Author

@RichardLindhout kindly accept the pull request. Thanks!

@iM-GeeKy
Copy link
Collaborator

I agree with you. @iM-GeeKy

Would you mind updating the docs as part of this pull request? I'll merge it afterward.

@SharfMohiuddin
Copy link
Contributor Author

sure @iM-GeeKy

@iM-GeeKy
Copy link
Collaborator

@SharfMohiuddin Thanks!

@iM-GeeKy iM-GeeKy merged commit ddbb824 into web-ridge:master Aug 28, 2023
5 checks passed
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.

3 participants