Skip to content

fix(ui): popup list controls overlap with table in list view #13967

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 3, 2025
Merged

fix(ui): popup list controls overlap with table in list view #13967

merged 1 commit into from
Oct 3, 2025

Conversation

kamal0620040
Copy link
Contributor

@kamal0620040 kamal0620040 commented Sep 30, 2025 •
edited
Loading

What?

  • Fixes an issue where the table header row was overlapping menu items in the ListControls component.

Why?

  • The overlapping occurred because the table lacked proper stacking context, causing z-index rules to affecting the visibility and usability of menu items in the ListControls component rendered above the table.

How?

  • Added the CSS property isolation: isolate to the table styles. This creates a new stacking context, ensuring that the table header does not interfere with overlapping UI elements

UI Changes:

Before:
image

After:
image

Fixes #13941

@PatrikKozak PatrikKozak self-requested a review October 3, 2025 13:52
@PatrikKozak PatrikKozak changed the title fix(ui): prevent header overlap by adding isolation to table styles fix(ui): popup list controls overlap with table in list view Oct 3, 2025
@PatrikKozak PatrikKozak merged commit 1e23882 into payloadcms:main Oct 3, 2025
184 of 187 checks passed
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 7, 2025

🚀 This is included in version v3.59.0

@JarrodMFlesch JarrodMFlesch mentioned this pull request Oct 14, 2025
Closed
@Crustan
Copy link

Crustan commented Oct 15, 2025

This is not fixed in v.3.59.1.

image

@ahmed-abdat ahmed-abdat mentioned this pull request Oct 15, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PatrikKozak PatrikKozak PatrikKozak approved these changes

@jacobsfletch jacobsfletch Awaiting requested review from jacobsfletch jacobsfletch is a code owner

@AlessioGr AlessioGr Awaiting requested review from AlessioGr AlessioGr is a code owner

@JarrodMFlesch JarrodMFlesch Awaiting requested review from JarrodMFlesch JarrodMFlesch is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

UI BUG With topbar of collections list

3 participants

@kamal0620040 @Crustan @PatrikKozak