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

[2.x] Fix overflow hidden issue with overlaping elements inside form section #668

Merged
merged 2 commits into from
Feb 9, 2021

Conversation

altercode
Copy link
Contributor

Hi,

I found issue with form section when it has small height and we have bigger elements inside it that overextends
the height of the form section. For example datapicker, dropdown etc. I attached screen below to show the issue.

My pull request fix that issue by slightly change the classes in form section components.

I have modified inertia and livewire component.

issue

@driesvints driesvints changed the title Fix overflow hidden issue with overlaping elements inside form section [2.x] Fix overflow hidden issue with overlaping elements inside form section Feb 9, 2021
@taylorotwell
Copy link
Member

Doesn't make sense to me to only have the top corners rounded when there are actions?

@taylorotwell
Copy link
Member

An type="date" input looks fine to me on Chrome...

image

@altercode
Copy link
Contributor Author

But for example custom datepicker has the behavior like on my screenshot. My pr remove the overflow-hidden from paren div and ads rounded corners at top left, top right and at the bottom of the form section. Even if there is no actions, rounded corners will be present at the bottom.

@taylorotwell
Copy link
Member

Got it.

@taylorotwell taylorotwell merged commit 1871fb4 into laravel:2.x Feb 9, 2021
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.

None yet

3 participants