fix(modal): fixed a bug that disallowed users to set the position of a modal

[rtraficante]

Description

The modal's position property is handled by the parent of the actual content box using flex, align-items, and justify-content but was blocked by the content box's property of "margin: auto" (m-auto). This fix deletes that property so the parent can correctly handle the position of the modal correctly.

Fixes #760

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

I tested this change by running the documentation site locally and checking whether the property works correctly on the Modal component page.

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules
• I have checked my code and corrected any misspellings

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
flowbite-react	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 30, 2023 11:00am

[codecov]

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (b472d85) 99.54% compared to head (6c04d0e) 99.54%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #766   +/-   ##
=======================================
  Coverage   99.54%   99.54%           
=======================================
  Files         129      129           
  Lines        6537     6537           
  Branches      391      391           
=======================================
  Hits         6507     6507           
  Misses         30       30           

Impacted Files	Coverage Δ
src/theme.ts	100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[tulup-conner]

Thank you! Can you rebase and move these changes to src/theme.ts? We just changed up our repository pretty substantially. You can see the new contributing guide here: https://github.com/themesberg/flowbite-react/blob/main/CONTRIBUTING.md

If you aren't comfortable rebasing, let me know and I'll be happy to do so for you.

[rtraficante]

@tulup-conner I'm like 96% sure I did that correctly

[Sagar301101]

Hey I have just used size props for modal

• for size sm, md and lg it is not working properly ( apart from this rest of size is working properly)
• I have used the latest of flowbite-react ( 0.5.0)
• please check this again