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

Modal outflows on top #537

Closed
peteracs opened this issue Jan 14, 2023 · 2 comments
Closed

Modal outflows on top #537

peteracs opened this issue Jan 14, 2023 · 2 comments
Labels
🐛 bug Something isn't working good first issue Good for newcomers

Comments

@peteracs
Copy link

peteracs commented Jan 14, 2023

Describe the bug
If the default modal has too much contents and the screen width is >=768 px, the modal outflows on the top, hiding the contents.

To Reproduce
Create a default modal, fill it up with a lot of content that should stretch the height of it. It will be okay with screen-width <768, not working =>768px

Screenshots

Here screen width is 767 px

Screenshot from 2023-01-14 00-39-31

Here screen width is 769 px

Screenshot from 2023-01-14 00-39-36

<div class="relative h-full w-full p-4 md:h-auto max-w-2xl">

md:h-auto I found it, this cause the issue.

@rluders rluders added the 🐛 bug Something isn't working label Jan 14, 2023
@rluders
Copy link
Collaborator

rluders commented Jan 14, 2023

Maybe it is related to #520 somehow?

@Allam0053
Copy link

I used this in my code

      <Modal
        className='!h-full !items-start'
      >
         // children
      </Modal>

modal overflow

basically, it overwrites the base styling 'items-center' and the '!h-full' just because I want to ensure that the height fills the screen.

this is only for overflowing elements, I didn't test it in non-overflowing elements

paghar pushed a commit to paghar/flowbite-react that referenced this issue Apr 25, 2023
@paghar paghar mentioned this issue Apr 25, 2023
15 tasks
rluders pushed a commit to paghar/flowbite-react that referenced this issue May 25, 2023
rluders pushed a commit to paghar/flowbite-react that referenced this issue May 25, 2023
rtraficante pushed a commit to rtraficante/flowbite-react that referenced this issue May 26, 2023
@rluders rluders closed this as completed Jul 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants