Skip to content

fix: remove the vertical scrolling on the home page; improve the profile page overflowing.#140

Merged
vansh-codes merged 1 commit into
vansh-codes:masterfrom
palchhinparihar:fix/overflow-on-mobile
Nov 15, 2025
Merged

fix: remove the vertical scrolling on the home page; improve the profile page overflowing.#140
vansh-codes merged 1 commit into
vansh-codes:masterfrom
palchhinparihar:fix/overflow-on-mobile

Conversation

@palchhinparihar

Copy link
Copy Markdown
Contributor

📄 Description

This PR focuses on fixing UI responsiveness issues and improving code structure.

  • 🐛 Fixed: Profile page overflow issues on both mobile and tablet screens.
  • 🐛 Fixed: Home page vertical scrolling issue.
  • Added: Disabled styling for the GitHub profile input when no profile link is entered.
  • Refactored: Split logic into separate .js files for better maintainability and replaced redundant code with cleaner implementations using the map() method.
  • 🏷️ Improved: Added title attributes to elements for better understanding of operations and accessibility.

🔗 Related Issues

Fixes #137


🖼️ Screenshots (if applicable)

Vertical Scrolling - Homepage

Screenshot 2025-11-05 164412

Overflow - Profile page (Mobile, Tablet, Laptop)

Screenshot 2025-11-05 164435 Screenshot 2025-11-05 164734 Screenshot 2025-11-05 164512

🧩 Type of Change

  • 🐛 Bug Fix
  • ✨ New Feature
  • ⚡ Enhancement / Optimization
  • 🧰 Refactoring
  • 🧾 Documentation Update
  • 🔧 Other (please specify): ____________

✅ Checklist

  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have added or updated relevant documentation.
  • My changes do not break any existing functionality.
  • I have tested my changes locally and they work as expected.
  • I have linked all relevant issues (if any).

💬 Additional Notes (Optional)

This PR enhances mobile and tablet responsiveness, improves maintainability, and makes the codebase more modular.

Let me know for any changes.

Thank you.

Copilot AI review requested due to automatic review settings November 5, 2025 11:28
@vercel

vercel Bot commented Nov 5, 2025

Copy link
Copy Markdown

@palchhinparihar is attempting to deploy a commit to the vansh-codes1's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

github-actions Bot commented Nov 5, 2025

Copy link
Copy Markdown
Contributor

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

Copilot AI 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.

Pull Request Overview

This PR refactors the preview card page to improve responsiveness and maintainability by extracting hardcoded configuration data into separate utility files and enhancing the UI with better mobile support.

  • Extracted export buttons, select dropdowns, and checkbox configurations into dedicated utility files
  • Enhanced responsive design with dynamic viewport heights, improved spacing, and mobile-friendly layouts
  • Added accessibility improvements with alt text, titles, and disabled state styling for the submit button

Reviewed Changes

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

Show a summary per file
File Description
src/pages/Home.jsx Updated viewport heights for responsive design, increased icon/input sizes, added disabled button state with visual feedback, and adjusted footer text color
src/app/[username]/page.jsx Refactored hardcoded UI elements to use imported configuration arrays, improved responsive grid layouts and width handling, added accessibility attributes (alt, title) to images and inputs
src/app/[username]/exportTools.js New file containing export button metadata (labels, icons, titles, actions)
src/app/[username]/selectTools.js New file containing select dropdown configurations (theme, font, pattern)
src/app/[username]/checkboxTools.js New file containing checkbox field configurations (star, fork, repo)
package-lock.json Dependency changes marking various packages as peer dependencies and removing dev-only flags

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

Comment thread src/app/[username]/selectTools.js
Comment thread src/app/[username]/page.jsx
@palchhinparihar

Copy link
Copy Markdown
Contributor Author

hello, @vansh-codes! are you there? it's been a few days. :)

@vansh-codes vansh-codes added wocs Contributors under WOC level 1 awards 2 points labels Nov 15, 2025
@vansh-codes vansh-codes merged commit b0266b9 into vansh-codes:master Nov 15, 2025
7 of 8 checks passed
@palchhinparihar palchhinparihar deleted the fix/overflow-on-mobile branch November 15, 2025 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

level 1 awards 2 points wocs Contributors under WOC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BUG: overflow UI on mobile screen size

3 participants