Skip to content

fix: connected the side filter bar to match the project list#215

Merged
agzaiyenth merged 1 commit into
IIT-SDGP:mainfrom
mdfaaeq:feature/project-section-enhancement
May 31, 2026
Merged

fix: connected the side filter bar to match the project list#215
agzaiyenth merged 1 commit into
IIT-SDGP:mainfrom
mdfaaeq:feature/project-section-enhancement

Conversation

@mdfaaeq
Copy link
Copy Markdown
Contributor

@mdfaaeq mdfaaeq commented May 26, 2026

Updated the project pages project list and the filter side bar to be in the same height

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 26, 2026

@mdfaaeq is attempting to deploy a commit to the Psycode Lab's Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adjusts the public project listing page so the filter sidebar and project list behave as same-height, independently scrollable panes.

Changes:

  • Added a fixed-height desktop two-column layout for filters and project results.
  • Updated infinite-scroll observation to use the nearest scroll container.
  • Replaced the next-page loading spinner with project-card skeleton placeholders.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
app/(public)/project/page.tsx Adds the two-pane scroll layout and global wheel redirection for the project list.
components/projects/project-explorer.tsx Updates infinite-scroll root handling and next-page loading UI.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread app/(public)/project/page.tsx
}
},
{ threshold: 0.1 }
{ threshold: 0.1, root: scrollRoot ?? null }
@agzaiyenth agzaiyenth merged commit b9d155c into IIT-SDGP:main May 31, 2026
0 of 2 checks passed
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.

3 participants