Skip to content

Add project filter feature#68

Open
Gunjan10-droid wants to merge 3 commits into
revatikadam0607:mainfrom
Gunjan10-droid:add-project-filter-feature
Open

Add project filter feature#68
Gunjan10-droid wants to merge 3 commits into
revatikadam0607:mainfrom
Gunjan10-droid:add-project-filter-feature

Conversation

@Gunjan10-droid

Copy link
Copy Markdown

Description

Implemented a dynamic project filtering feature to improve portfolio navigation and user experience.

Features Added

  • Added category filter buttons
  • Dynamic project filtering without page reload
  • Active filter highlight state
  • Responsive filter UI
  • Multiple category support for projects

Changes Made

  • Updated projects.json with project categories
  • Added filter buttons in index.html
  • Implemented filtering logic in main.js
  • Added responsive styling in style.css

issue #58

@Gunjan10-droid

Copy link
Copy Markdown
Author

@revatikadam0607 I have solved the issue. Would you merge it and also label the issue for nsoc?

@revatikadam0607 revatikadam0607 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Hi,

Thank you for raising the PR and contributing to the project. I apologize for not noticing your PR earlier from my side.

I recently made some updates to the project, so the current codebase has changed a bit since the issue was opened. Would it be possible for you to update your changes based on the latest version of the code and then update the PR accordingly?

I really appreciate your contribution and the effort you've put into this. Please let me know if you need any help with syncing the latest changes.

Thank you!

@Gunjan10-droid

Copy link
Copy Markdown
Author

Hi @revatikadam0607,

I've updated my branch with the latest changes from the current main branch and pushed the updates to this PR. GitHub now shows that there are no conflicts with the base branch.

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.

2 participants