Skip to content

Implemented a “Show More / Show Less” feature for skill suggestion chips on the homepage.#288

Closed
Pranathi-Kunjeti wants to merge 4 commits into
komalharshita:mainfrom
Pranathi-Kunjeti:feat/limit-skill-suggestions
Closed

Implemented a “Show More / Show Less” feature for skill suggestion chips on the homepage.#288
Pranathi-Kunjeti wants to merge 4 commits into
komalharshita:mainfrom
Pranathi-Kunjeti:feat/limit-skill-suggestions

Conversation

@Pranathi-Kunjeti

Copy link
Copy Markdown

Summary [required]

This PR improves the skill suggestion section on the homepage by limiting the number of visible quick-select skill chips initially and adding a “Show More / Show Less” toggle button.

The update reduces UI clutter on smaller screens and improves overall readability and responsiveness without affecting the existing skill selection functionality

Related Issue [required]

Closes #195

Type of Change [required]

  • Bug fix — resolves a broken behaviour
  • Feature — adds new functionality
  • Data — adds new projects to data/projects.json
  • Documentation — updates docs, README, or code comments only
  • Style — CSS or visual changes only, no logic change
  • Refactor — restructures code without changing behaviour
  • Test — adds or updates tests

What Was Changed [required]

File Change made
templates/index.html Added Show More / Show Less toggle button and grouped extra skill chips
static/style.css Added styles for hidden extra skills and toggle button
static/script.js Added toggle functionality for expanding/collapsing extra skill suggestions

How to Test This PR [required]

  1. Clone this branch: git checkout feat/limit-visible-skill-suggestions
  2. Install dependencies: pip install -r requirements.txt
  3. Run the app: python app.py
  4. Open http://127.0.0.1:5000
  5. Run the tests: python tests/test_basic.py

Expected test output:

30 passed, 0 failed out of 30 tests

Test Results [required]

29 passed, 1 failed out of 30 tests
 Failing test: FAIL test_health_check: test_health_check() missing 1 required positional argument: 'client'

Screenshots (if UI change)

Before :
Screenshot 2026-05-18 163343

After:
Screenshot 2026-05-18 163409

Screenshot 2026-05-18 163418

Self-Review Checklist [required]

  • I have read CONTRIBUTING.md and followed all guidelines
  • My branch name follows the convention: feat/, fix/, docs/, data/, style/, test/
  • I have run python tests/test_basic.py and all 29 tests pass
  • I have run flake8 . locally and there are no errors
  • I have not introduced any print() or console.log() debug statements
  • Every new function I wrote has a docstring
  • I have not modified files outside the scope of the linked issue
  • If I changed the UI, I tested it at 375px (mobile) and 1280px (desktop)
  • If I added a project to the dataset, it has all required JSON fields

Notes for Reviewer

One existing test (test_health_check) is currently failing due to a missing required positional argument (client). This issue is unrelated to the changes introduced in this PR.

@vercel

vercel Bot commented May 18, 2026

Copy link
Copy Markdown

@Pranathi-Kunjeti is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thank you for submitting your first pull request to DevPath.

Before review:

  • Complete the PR template fully
  • Ensure all tests pass
  • Link your PR to an issue
  • Keep changes scoped to the issue

A maintainer will review your contribution soon.

@Pranathi-Kunjeti

Copy link
Copy Markdown
Author

please have a look at my pull request @komalharshita

@komalharshita

Copy link
Copy Markdown
Owner

closing this PR

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

Labels

gssoc:spam This will not be worked on gssoc-2026

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Limit visible skill suggestions with “Show More” option

2 participants