Skip to content

fix:mainpage and project page styles fixed#346

Open
Saqlain532 wants to merge 1 commit into
komalharshita:mainfrom
Saqlain532:feat/main-project-page-styles-fixed
Open

fix:mainpage and project page styles fixed#346
Saqlain532 wants to merge 1 commit into
komalharshita:mainfrom
Saqlain532:feat/main-project-page-styles-fixed

Conversation

@Saqlain532
Copy link
Copy Markdown

Summary [required]

This PR improves UI consistency across DevPath and also fixes a critical issue in script.js where project recommendations were not generating correctly. It resolves broken skill selection behavior, search handling, and ensures the recommendation system correctly builds payloads and returns results. Along with this, navbar and footer inconsistencies across pages have been fixed, and GitHub icon styling has been improved for better UI consistency.

Related Issue [required]

Closes #

Type of Change [required]

  • Bug fix — fixes broken project generation and UI issues
  • Style — CSS/UI improvements
  • Refactor — improves frontend structure and behavior

What Was Changed [required]

File | Change made -- | -- script.js | Fixed project generation logic, skill selection handling, and search flow CSS/navbar.css | Fixed navbar alignment and GitHub icon styling CSS/footer.css | Fixed footer consistency across pages CSS/form.css | Fixed broken form layout and spacing issues CSS/skill-strip.css | Fixed UI inconsistencies in skill strip Project page template | Fixed navbar and footer mismatch

How to Test This PR [required]

  1. Checkout branch:

    git checkout feat/main-project-page-styles-fixed
  2. Run project:

    python app.py
  3. Open:

    http://127.0.0.1:5000
  4. Test:
    • Add skills and submit form
    • Verify project recommendations are generated correctly
    • Test quick-pick skills and manual input
    • Check navbar/footer consistency across pages
    • Verify GitHub icon visibility
    • Test responsive layout

Test Results [required]

30 passed, 0 failed out of 30 tests

Screenshots (if UI change)

Before : Screenshot 2026-05-20 072411

.
Screenshot 2026-05-20 072404
Screenshot 2026-05-20 072352
Screenshot 2026-05-20 064622
Screenshot 2026-05-20 064612
After :
Screenshot 2026-05-20 100026

Screenshot 2026-05-20 095951 Screenshot 2026-05-20 095936 Screenshot 2026-05-20 095927

Self-Review Checklist [required]

  • Followed CONTRIBUTING guidelines
  • Branch naming convention followed
  • All tests passed locally
  • No debug logs left in code
  • Changes are within issue scope
  • UI tested on mobile and desktop
  • Verified project generation works correctly after fix

Notes for Reviewer

This PR includes both UI improvements and a critical fix in script.js that restores project recommendation generation functionality. No backend changes were made.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 20, 2026

@Saqlain532 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.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

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.

@Saqlain532
Copy link
Copy Markdown
Author

I have fixed all broken issues that is currently open because i was assigned to fix styling of project page but main page was not working which leads me to fix all issues on the main page please review my PR i have also break long CSS file into components for easy debugging the code

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant