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

Fix padded appearance near sub-navigation select element #15030

Merged
merged 1 commit into from
Dec 9, 2024

Conversation

Persaeus
Copy link
Contributor

@Persaeus Persaeus commented Dec 9, 2024

Description

With 1e65d98, on mobile the sub-navigation select element appears to have padding above and below when the render hooks for PAGE_SUB_NAVIGATION_SELECT are empty.

This is because the wrappers for the hooks, even when empty, are affected by the flexbox gap.

This PR adds display: contents to each to undo this effect.

Visual changes

Before After
before after

Functional changes

  • Code style has been fixed by running the composer cs command.
  • Changes have been tested to not break existing functionality.
  • Documentation is up-to-date.

@danharrin
Copy link
Member

Very nice fix, thanks

@danharrin danharrin added bug Something isn't working ui labels Dec 9, 2024
@danharrin danharrin added this to the v3 milestone Dec 9, 2024
@danharrin
Copy link
Member

Fixes #14713

@danharrin danharrin merged commit 81b06a6 into filamentphp:3.x Dec 9, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ui
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants