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: set proper default value in radix with content block #4937

Merged
merged 2 commits into from
Mar 4, 2025

Conversation

TrySound
Copy link
Member

@TrySound TrySound commented Mar 2, 2025

User reported in discord.

Radix components like accordion does not open first item by default when use with content block. The reason was in our default detection.

We used indexes within ancestor and when put item inside of content block template it gets 0 but not rendered.

Here improved the logic to reset index when render inside of block template.

Screenshot 2025-03-01 at 21 14 09

User reported in discord.

Radix components like accordion does not open first item by default when
use with content block. The reason was in our default detection.

We used indexes within ancestor and when put item inside of content
block template it gets 0 but not rendered.

Here improved the logic to reset index when render inside of block
template.
@TrySound TrySound requested review from kof and johnsicili March 2, 2025 06:53
@TrySound TrySound merged commit bdf4463 into main Mar 4, 2025
17 checks passed
@TrySound TrySound deleted the content-block branch March 4, 2025 11:26
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

2 participants