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

update : Updated showTopModalSheet to include borderRadius customization #6

Merged
merged 1 commit into from
Aug 11, 2023

Conversation

iamthejahid
Copy link
Contributor

PR Title: Add radius support for TopModalSheet

Description:

This PR addresses issue #5 by adding support for customizing the border radius of the TopModalSheet function. It introduces a new optional borderRadius parameter that allows developers to specify the desired border radius for the modal sheet.

Changes Made:

  • Added a new borderRadius parameter to the showTopModalSheet function.
  • Updated the showTopModalSheet implementation to handle the borderRadius parameter and apply the specified border radius to the modal sheet.
  • Included a usage example showcasing the usage of the borderRadius parameter.

Why:

The motivation behind this change is to provide more flexibility and customization options for developers using the TopModalSheet function. By allowing the border radius to be adjusted, developers can create modal sheets that better align with their app's design aesthetic.

Result:

By incorporating this enhancement, developers can easily create TopModalSheets with custom border radius, enhancing the visual appeal and user experience of their Flutter apps.

Please review and merge this PR at your convenience. Thank you!

@Pilaba
Copy link
Owner

Pilaba commented Aug 11, 2023

LGTM 😎

@Pilaba Pilaba merged commit c4a2e17 into Pilaba:master Aug 11, 2023
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