Skip to content

fix: UI Improvement in /devcard-demo - Fix Return Home button overlap#182

Closed
anuragbraveboy-sudo wants to merge 1 commit into
Dev-Card:mainfrom
anuragbraveboy-sudo:fix/return-home-button-overlap
Closed

fix: UI Improvement in /devcard-demo - Fix Return Home button overlap#182
anuragbraveboy-sudo wants to merge 1 commit into
Dev-Card:mainfrom
anuragbraveboy-sudo:fix/return-home-button-overlap

Conversation

@anuragbraveboy-sudo

@anuragbraveboy-sudo anuragbraveboy-sudo commented May 19, 2026

Copy link
Copy Markdown
Contributor

Fixes #121

Problem

The /devcard-demo route did not exist, which meant the Return Home
The button had no proper page structure — causing it to overlap with
main content, especially on smaller screen sizes.

Root Cause

No /devcard-demo page existed in the codebase. The button had no
parent layout constraints, leading to overlap and broken UI.

Solution

✅ Created the missing /devcard-demo route with +page.svelte
✅ Used flexbox layout (flex-direction: column) so Return Home
button stays in normal document flow — no overlap possible
✅ Added top-bar div with proper padding and border separator
✅ Dark background (#0f1222) matching existing DevCard styling
✅ Responsive media queries for all breakpoints:

  • 1024px — adjusted content padding
  • 768px — reduced top-bar and content padding
  • 480px — smaller button font and padding for mobile

Screenshots

Before Fix:

  • /devcard-demo route did not exist — button had no proper styling or layout
Before Fix

After Fix:

  • Return Home button clearly visible at top ✅
  • Zero overlap with main content ✅
  • Page renders correctly at /devcard-demo
Screenshot 2026-05-21 222955

Testing

  • pnpm lint → web app has no lint script (expected per CONTRIBUTING.md)
  • pnpm test → web app has no test script (expected per CONTRIBUTING.md)
  • Tested on desktop, tablet, and mobile viewports ✅

@ShantKhatri

Copy link
Copy Markdown
Contributor

Hi @anuragbraveboy-sudo this is our current demo page.
image

The one for which ss you have mentioned is totally different. Add on to this, if you have created seperate new page for the demo, then how come the current page's implementation is not yet removed?

@ShantKhatri ShantKhatri self-requested a review May 20, 2026 09:36
@anuragbraveboy-sudo

Copy link
Copy Markdown
Contributor Author

Hi @ShantKhatri, you were right! I apologise for the confusion.

I've now fixed the Return Home button directly in the u/[username] page (the actual "Profile not found" state). The button is now moved to a dedicated top-bar div at the top of the page — completely outside the error card, so there's zero overlap.

I have updated the pr description you see check the SCREENSHOT over there .Please let me know if any further changes are needed!

@anuragbraveboy-sudo anuragbraveboy-sudo force-pushed the fix/return-home-button-overlap branch from 985f706 to 2e3eaa6 Compare May 21, 2026 17:25
@ShantKhatri

Copy link
Copy Markdown
Contributor

Hi @ShantKhatri, you were right! I apologise for the confusion.

I've now fixed the Return Home button directly in the u/[username] page (the actual "Profile not found" state). The button is now moved to a dedicated top-bar div at the top of the page — completely outside the error card, so there's zero overlap.

I have updated the pr description you see check the SCREENSHOT over there .Please let me know if any further changes are needed!

Please add SS or demo to the new changes.

@ShantKhatri

Copy link
Copy Markdown
Contributor

Hi @anuragbraveboy-sudo , as we have moved out of Svelt and migrated to React, it's recommended to have changes aligned with the new architecture. Happy to review your updated PR. Closing this.

@ShantKhatri ShantKhatri closed this Jun 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

design gssoc:approved Required label for every approved PR. Gives the base +50 points and enables contribution tracking.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI IMPROVEMENT IN /devcard-demo

3 participants