Skip to content

fix(web-app): fix UI grid distortion, local logo asset route and project sorting#432

Merged
steam-bell-92 merged 2 commits into
steam-bell-92:mainfrom
nishtha-agarwal-211:fix/ui-distortion-sorting-419
May 19, 2026
Merged

fix(web-app): fix UI grid distortion, local logo asset route and project sorting#432
steam-bell-92 merged 2 commits into
steam-bell-92:mainfrom
nishtha-agarwal-211:fix/ui-distortion-sorting-419

Conversation

@nishtha-agarwal-211
Copy link
Copy Markdown
Contributor

Description

This pull request addresses Issue #419 to resolve visual distortion on the main page grid, fix category page icon styling, and enforce alphabetical ordering across project catalogs.

Key Changes

✅ Fixed Grid Layout Distortion (index.html)

  • Restored missing wrapper containers:
    • .projects-section
    • .container
    • .projects-grid
  • Prevented project cards from stretching to full width.
  • Re-established proper responsive grid alignment.

✅ Implemented Alphabetical Card Sorting

Files updated:

  • index.html
  • games.html
  • math.html
  • utilities.html

Changes:

  • Sorted all project cards alphabetically.
  • Removed duplicate "Fibonacci Series" entry from the homepage.

✅ Fixed Logo Asset Pathing

Files updated:

  • index.html
  • assets/logo.png

Changes:

  • Added local assets/logo.png.
  • Updated image source from:
    /python-mini-project/images/logo.png

Copilot AI review requested due to automatic review settings May 19, 2026 14:44
@vercel
Copy link
Copy Markdown

vercel Bot commented May 19, 2026

@nishtha-agarwal-211 is attempting to deploy a commit to the Anuj's projects 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 updates the web-app UI to correct layout issues on the homepage, switch the navbar logo to a local asset, and reorder project cards alphabetically across the main and category pages.

Changes:

  • Restored the homepage projects grid wrapper structure and reordered cards alphabetically (also removing a duplicate Fibonacci entry).
  • Updated the navbar logo to use the local assets/logo.png path.
  • Updated global .card-icon styling so icons render consistently (not hidden).

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
web-app/index.html Fixes homepage grid structure, reorders cards alphabetically, and updates the logo image path.
web-app/games.html Reorders game cards alphabetically (but introduces a broken data-project key for 2048).
web-app/math.html Reorders math cards alphabetically.
web-app/utilities.html Reorders utilities cards alphabetically.
web-app/css/styles.css Makes .card-icon visible and styles it as a consistent icon container.

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

Comment thread web-app/games.html Outdated
Comment thread web-app/index.html Outdated
Comment thread web-app/math.html
Comment thread web-app/utilities.html
@steam-bell-92 steam-bell-92 merged commit 98b69ad into steam-bell-92:main May 19, 2026
2 of 3 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

🎉 Thank you for your contribution!

Your Pull Request has been merged successfully.

We appreciate the time and effort you put into improving this project. Contributions like yours help the repository grow and stay useful for everyone.

If you'd like to contribute again, please check the open issues and make sure you are assigned before opening another Pull Request.

Thanks again for your support! 🙌

@steam-bell-92
Copy link
Copy Markdown
Owner

@nishtha-agarwal-211 Great work with that execution

@nishtha-agarwal-211
Copy link
Copy Markdown
Contributor Author

@nishtha-agarwal-211 Great work with that execution

Thank you! Glad to contribute 🙌

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.

3 participants