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

Filter PopOver goes off screen in Mobile devices (sm and small under 450px) #3418

Closed
lucidpolygon opened this issue Aug 7, 2022 · 3 comments · Fixed by #3443
Closed

Filter PopOver goes off screen in Mobile devices (sm and small under 450px) #3418

lucidpolygon opened this issue Aug 7, 2022 · 3 comments · Fixed by #3443
Labels
bug Something isn't working confirmed

Comments

@lucidpolygon
Copy link

lucidpolygon commented Aug 7, 2022

Package

filament/tables

Package Version

2.15.3

Laravel Version

9.22.1

Livewire Version

2.10

PHP Version

8.1.4

Problem description

On mobile screens, the Popover filter div is going off-screen. Most likely caused by the left margin.

Expected behavior

The popOver filter must not overflow

Steps to reproduce

  1. Attached is a simple project
  2. Connect SQLite database
  3. Run the project and check the home page in mobile view
  4. Try to filter

Reproduction repository

https://github.com/lucidpolygon/filament-asda

Relevant log

Screen Shot 2022-08-06 at 5 16 43 PM

output

No response

@zepfietje
Copy link
Member

This is most probably caused by Alpine Floating UI. I'm reworking dropdowns in #3443 and have briefly looked into this already.

I think we can close this issue, since it will definitely be tested (and thus fixed) before that PR is merged, @danharrin?

@danharrin
Copy link
Member

We'll just link the PR

@danharrin danharrin linked a pull request Aug 19, 2022 that will close this issue
3 tasks
@danharrin
Copy link
Member

Fixed by #3443

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working confirmed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants