Skip to content

fix(ui): remove duplicate mobile Discord button and improve drawer UX#309

Draft
RedStar071 wants to merge 2 commits into
mainfrom
cursor/fix-mobile-header-nav-1124
Draft

fix(ui): remove duplicate mobile Discord button and improve drawer UX#309
RedStar071 wants to merge 2 commits into
mainfrom
cursor/fix-mobile-header-nav-1124

Conversation

@RedStar071

@RedStar071 RedStar071 commented Jul 22, 2026

Copy link
Copy Markdown
Member

Summary

  • Hide header Sign in / Add App controls below md so the open mobile menu no longer shows a duplicate Discord icon next to the close button
  • Replace the full-width elongated Sign in button with a compact rounded-lg control in the mobile drawer
  • Improve mobile menu UX: larger tap targets, clearer nested link hierarchy, sticky action footer, Add App CTA, and working Features destinations

Test plan

  • Open the site on a narrow viewport and open the hamburger menu
  • Confirm only the close (X) control appears in the top-right (no Discord icon)
  • Confirm Sign in is compact (not a full-width pill)
  • Confirm Features / Applications children are tappable and Add App appears when an invite URL exists
  • Confirm desktop header still shows Add App + Sign in from md up

Verification

  • pnpm lint:fix — clean
  • pnpm typecheck — pass
  • pnpm test:unit — pass
  • Header-related Nuxt tests — 12/12 pass
  • pnpm build — pass (with test secrets + site URL)
Open in Web Open in Cursor 

View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

Confidence Score: 4/5

Feature navigation from non-WolfStar routes needs a fix before merging.

The new hash-only links work on the WolfStar landing page. The shared header also exposes them on pages without the target elements, where they do not navigate or scroll. No security issues were found in the changed code.

app/composables/useHeader.ts

T-Rex T-Rex Logs

What T-Rex did

  • T-Rex attempted a production-build verification, which was blocked when the build terminated with exit code 137 and failed to produce Nuxt server output.
  • A bounded Nuxt development-server fallback was started and polled twelve times but never became reachable at /staryl due to resource exhaustion from concurrent repository build and development processes.
  • T-Rex completed the verification pass but reported that its local artifact references were not uploaded.

View all artifacts

T-Rex Ran code and verified through T-Rex

Fix All in Claude Code Fix All in Cursor Fix All in Cursor Cloud Agents

Prompt To Fix All With AI
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
app/composables/useHeader.ts:70
**Hash Links Retain Wrong Route**

The shared header appears on routes such as `/staryl`, `/blog`, and `/login`, but `to="#moderation-tools"` retains the current pathname. Those pages do not contain this section, so tapping the new menu item changes only the hash and does not navigate or scroll to the requested feature; the three links should include the WolfStar route.

Reviews (1): Last reviewed commit: "[autofix.ci] apply automated fixes" | Re-trigger Greptile

Greptile also left 1 inline comment on this PR.

cursoragent and others added 2 commits July 22, 2026 14:39
Keep Sign in / Add App in the header for desktop only, and use a compact
rounded Sign in control plus clearer spacing and destinations in the
mobile navigation drawer.

Co-authored-by: RedStar <redstar071@proton.me>
@codspeed-hq

codspeed-hq Bot commented Jul 22, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 44 untouched benchmarks


Comparing cursor/fix-mobile-header-nav-1124 (a8810c7) with main (6194ef0)

Open in CodSpeed

{
description: "Tools to help you moderate your server",
label: "Moderation Tools",
to: "#moderation-tools",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Hash Links Retain Wrong Route

The shared header appears on routes such as /staryl, /blog, and /login, but to="#moderation-tools" retains the current pathname. Those pages do not contain this section, so tapping the new menu item changes only the hash and does not navigate or scroll to the requested feature; the three links should include the WolfStar route.

Context Used: CLAUDE.md (source)

Prompt To Fix With AI
This is a comment left during a code review.
Path: app/composables/useHeader.ts
Line: 70

Comment:
**Hash Links Retain Wrong Route**

The shared header appears on routes such as `/staryl`, `/blog`, and `/login`, but `to="#moderation-tools"` retains the current pathname. Those pages do not contain this section, so tapping the new menu item changes only the hash and does not navigate or scroll to the requested feature; the three links should include the WolfStar route.

**Context Used:** CLAUDE.md ([source](https://app.greptile.com/wolfstar-project/github/wolfstar-project/wolfstar.rocks/-/custom-context?memory=d7db7ccf-40f4-43f6-9080-ac8602584fb3))

How can I resolve this? If you propose a fix, please make it concise.

Fix in Claude Code Fix in Cursor Fix in Cursor Cloud Agents

@cursor cursor Bot changed the title fix(header): remove duplicate mobile Discord button and improve drawer UX fix(ui): remove duplicate mobile Discord button and improve drawer UX Jul 22, 2026
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.

2 participants