Skip to content

style: improve integration category chips#7596

Open
ankitkr104 wants to merge 1 commit intolayer5io:masterfrom
ankitkr104:issue-7593-improve-integrations-category-styling
Open

style: improve integration category chips#7596
ankitkr104 wants to merge 1 commit intolayer5io:masterfrom
ankitkr104:issue-7593-improve-integrations-category-styling

Conversation

@ankitkr104
Copy link
Copy Markdown

@ankitkr104 ankitkr104 commented Apr 8, 2026

Description

This PR fixes #7593

Notes for Reviewers
While working on this project, I faced significant challenges in setting up the development environment locally. The project requires high system resources, and even after attempting multiple optimizations (lightweight build, reducing collections, limiting memory usage, etc.), the development server frequently failed to run due to memory constraints.

Because of this, I was not able to fully run the project on localhost. However, I carefully understood the codebase, analyzed the relevant components, and made the changes accordingly.

Request for Review

I kindly request the maintainers to review my changes. It would be very helpful if you could verify the functionality and provide feedback.

Signed commits

  • Yes, I signed my commits.

@arjunmehta-git
Copy link
Copy Markdown

Good work addressing the category chip styling, @ankitkr104. The approach of updating Integration.style.js alongside the grid component is correct — keeping style logic in the styled-component file rather than inline is the right pattern for this codebase.

One thing I want to flag from your notes:

I was not able to fully run the project on localhost due to memory constraints

This is understandable — the full build is resource-intensive. That's exactly why we have lite build profiles (make site / LITE_BUILD_PROFILE=core). For a styling change like this, the core or content profile should be sufficient to exercise the integrations page. Please give it a try with make site and attach a screenshot of the chip rendering in both light and dark mode to the PR.

Without a visual confirmation, it's difficult for reviewers to verify the intent. A before/after screenshot from the Netlify preview would also work once the preview build completes.

Once we can see the chips rendering as intended, this should be straightforward to approve.

@ankitkr104
Copy link
Copy Markdown
Author

ankitkr104 commented Apr 9, 2026

Hey @arjunmehta-git
thanks for your understanding

See what and why i am facing these issues from past 2-3 days

On Windows:

make site fails because Windows doesn't have make command by default
You need either WSL or Git Bash to make it work

What’s happening:

WSL is installed, but Ubuntu cannot start because its main virtual disk file is missing in my PC.
That file is the one WSL uses to run Linux.
Because of that, make site cannot run inside Ubuntu.
Trying wsl --install does not fix it, because Windows says Ubuntu already exists, so it stops instead of repairing it.

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.

Improve styling of categories of models

2 participants