Skip to content

fix: resolve uneven margin caused by card flex overflow#3604

Open
Iam-Karan-Suresh wants to merge 5 commits intokestra-io:mainfrom
Iam-Karan-Suresh:fix/blueprint-card-margin-overflow
Open

fix: resolve uneven margin caused by card flex overflow#3604
Iam-Karan-Suresh wants to merge 5 commits intokestra-io:mainfrom
Iam-Karan-Suresh:fix/blueprint-card-margin-overflow

Conversation

@Iam-Karan-Suresh
Copy link
Contributor

@Iam-Karan-Suresh Iam-Karan-Suresh commented Jan 12, 2026

Closes #3602.

What was fixed

I fixed an uneven margin issue on this page which was causing visual misalignment.

The Explore Blueprints carousel showed uneven right-side spacing and card misalignment after navigating with the arrow buttons, especially on smaller screen sizes.

Root cause

The carousel scroll distance was hardcoded, causing cards to overshoot the visible area. In addition, horizontal padding and implicit margins contributed to the appearance of extra space on the right.

Solution

  • Fixed the margin around the cards with responsive
  • Replaced fixed scroll values with a dynamic scroll calculation based on actual card width and gap
  • Added scroll snapping to ensure cards align cleanly after navigation
  • Removed unintended horizontal padding from container sections
  • Explicitly reset carousel margins to prevent layout overflow

Result

  • Cards remain properly aligned
  • No extra right-side spacing or horizontal overflow
  • Improved responsive behavior and visual consistency

Screenshots

image

Screen Record

Screencast.from.2026-01-12.23-08-18.mp4

@github-project-automation github-project-automation bot moved this to To review in Pull Requests Jan 12, 2026
@MilosPaunovic MilosPaunovic added area/frontend Needs frontend code changes kind/external Pull requests raised by community contributors labels Jan 13, 2026
@Piyush-r-bhaskar
Copy link
Contributor

To be reviewed after migration to Astro.

@tchiotludo tchiotludo force-pushed the main branch 7 times, most recently from e7af5ef to cee7be0 Compare January 22, 2026 13:49
@Iam-Karan-Suresh
Copy link
Contributor Author

Understood, thanks for letting me know.
I’ll wait for the review after the Astro migration.

@Piyush-r-bhaskar
Copy link
Contributor

Hey @Iam-Karan-Suresh

Can you fix the conflict before I take a look ?

@tchiotludo tchiotludo force-pushed the main branch 2 times, most recently from 870cbbd to 3ad7495 Compare January 24, 2026 11:41
@MilosPaunovic
Copy link
Member

Just a small note for the future, if you could use GitHub Keywords when opening PRs to connect it to the proper issue (to automate the entire process a bit more), it would be super helpful to us. You can read a bit more about it here (I've added closing keyword for this particular issue).

@MilosPaunovic
Copy link
Member

Hey @Iam-Karan-Suresh, are there any updates on this?

@Iam-Karan-Suresh Iam-Karan-Suresh force-pushed the fix/blueprint-card-margin-overflow branch from 082dd80 to 6b9eea8 Compare January 27, 2026 12:42
Signed-off-by: Iam-karan-suresh <karansuresh.info@gmail.com>
@Iam-Karan-Suresh Iam-Karan-Suresh force-pushed the fix/blueprint-card-margin-overflow branch from 6b9eea8 to 839ed30 Compare January 27, 2026 13:48
@Iam-Karan-Suresh
Copy link
Contributor Author

Iam-Karan-Suresh commented Jan 27, 2026

Hey @MilosPaunovic @Piyush-r-bhaskar , sorry for the late reply 🙏
I’ve resolved the conflicts and pushed the updated changes.

Thanks for the note about GitHub keywords — I’ll make sure to use them in future PRs 👍

@MilosPaunovic MilosPaunovic removed the kind/do-not-merge Don't merge label Jan 27, 2026
Copy link
Contributor

@Piyush-r-bhaskar Piyush-r-bhaskar left a comment

Choose a reason for hiding this comment

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

Hey @Iam-Karan-Suresh

Can you please fix the x-axis overflow ?

Image

@Piyush-r-bhaskar Piyush-r-bhaskar moved this from To review to On hold in Pull Requests Feb 2, 2026
@MilosPaunovic
Copy link
Member

Hey @Iam-Karan-Suresh, are there any updates on this?

@Iam-Karan-Suresh
Copy link
Contributor Author

Hi @MilosPaunovic I'll push the changes with 3 hours. Thank you for your patience.

Signed-off-by: Iam-karan-suresh <karansuresh.info@gmail.com>
@MilosPaunovic MilosPaunovic moved this from On hold to To review in Pull Requests Feb 3, 2026
@Iam-Karan-Suresh
Copy link
Contributor Author

Screenshot from 2026-02-03 11-23-57

this is the output example for the x-axis overflow

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/frontend Needs frontend code changes kind/external Pull requests raised by community contributors

Projects

Status: To review

Development

Successfully merging this pull request may close these issues.

Bad margin on responsive design on data enginneering

3 participants