Skip to content

fix: restore vertical scroll for Query Loop carousel#104

Open
Sukhendu2002 wants to merge 1 commit intortCamp:developfrom
Sukhendu2002:fix/vertical-query-loop-scroll
Open

fix: restore vertical scroll for Query Loop carousel#104
Sukhendu2002 wants to merge 1 commit intortCamp:developfrom
Sukhendu2002:fix/vertical-query-loop-scroll

Conversation

@Sukhendu2002
Copy link
Copy Markdown
Collaborator

Fixes: #103

Summary

When a Carousel block is configured with Orientation: Vertical and uses a Query Loop block as its content source, the previous/next navigation buttons are disabled, and mouse drag scrolling does not work.

Type of change

  • Bug fix
  • New feature
  • Enhancement/refactor
  • Documentation update
  • Test update
  • Build/CI/tooling

What changed

  • Extended vertical-axis container rule to also target .wp-block-post-template so Embla receives flex-direction: column when Query Loop is active
  • Added height: 100% to .wp-block-query for vertical axis to complete the CSS height chain between .embla and .wp-block-post-template
  • Added max-height: var(--carousel-kit-slide-width, 100%) to slides in vertical mode to mirror the existing max-width on the cross axis

Breaking changes

Does this introduce a breaking change? If yes, describe the impact and migration path below.

  • Yes — migration path:
  • No

Testing

Describe how this was tested.

  • Unit tests
  • Manual testing
  • Cross-browser testing (if UI changes)

Test details:

Screenshots / recordings

Before:

Screen.Recording.2026-04-02.at.3.39.31.PM.mov

After

Screen.Recording.2026-04-02.at.3.40.35.PM.mov

Checklist

  • I have self-reviewed this PR
  • I have added/updated tests where needed
  • I have updated docs where needed
  • I have checked for breaking changes

@Sukhendu2002 Sukhendu2002 marked this pull request as ready for review April 2, 2026 10:25
@theMasudRana theMasudRana self-requested a review April 2, 2026 19:54
Copy link
Copy Markdown
Collaborator

@theMasudRana theMasudRana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good fix.
I have tested on my end as well, and it's working as expected.

Thanks.

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.

[Bug]: Vertical carousel with Query Loop nav buttons disabled and drag non-functional

2 participants