Skip to content

fix: prevent contained styles from leaking into nested RepeatableEntry items - #19297

Merged
danharrin merged 1 commit into
filamentphp:4.xfrom
people-sea:fix/issues/19285
Feb 19, 2026
Merged

fix: prevent contained styles from leaking into nested RepeatableEntry items#19297
danharrin merged 1 commit into
filamentphp:4.xfrom
people-sea:fix/issues/19285

Conversation

@people-sea

Copy link
Copy Markdown
Member

Description

Fixes: #19285

Use direct child selector to prevent nested RepeatableEntry from inheriting parent's contained styling.

Visual changes

before

image

after

image

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.

@webard

webard commented Feb 19, 2026

Copy link
Copy Markdown
Contributor

I think that might have been the intended behavior. After this PR there is no way to distinguish these repeaters.

@people-sea

people-sea commented Feb 19, 2026

Copy link
Copy Markdown
Member Author

I think that might have been the intended behavior. After this PR there is no way to distinguish these repeaters.

@webard Thanks for the feedback! However, the reverse case already works --- child contained(true) inside parent contained(false) renders correctly. This asymmetry suggests it's a selector bug rather than intended behavior.

@people-sea

people-sea commented Feb 19, 2026

Copy link
Copy Markdown
Member Author

That said, the lack of visual separation between items when contained(false) is used could be worth a separate UX improvement.

@danharrin danharrin added the bug Something isn't working label Feb 19, 2026
@danharrin danharrin added this to the v4 milestone Feb 19, 2026
@danharrin
danharrin merged commit fa5efdf into filamentphp:4.x Feb 19, 2026
24 checks passed
@people-sea
people-sea deleted the fix/issues/19285 branch February 27, 2026 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Contained function doesnt work in nested RepeatableEntry

3 participants