Skip to content

Move Home/Leaderboard nav icons to top bar; persist PoFunQuiz title in header#1

Draft
Copilot wants to merge 2 commits intomasterfrom
copilot/move-icons-to-top-nav
Draft

Move Home/Leaderboard nav icons to top bar; persist PoFunQuiz title in header#1
Copilot wants to merge 2 commits intomasterfrom
copilot/move-icons-to-top-nav

Conversation

Copy link

Copilot AI commented Mar 1, 2026

Bottom nav was the sole navigation surface, mixing app-level links (Home, Leaderboard) with a debug link (Diag). Moves the primary nav icons to the top header and removes the bottom nav bar entirely.

Changes

  • MainLayout.razor — Header now always renders PoFunQuiz title (left) + Home/Leaderboard NavLink icons (right) directly. Removed SectionOutlet and bottom <NavMenu />.
  • NavMenu.razor — Stripped to a single Diag top-nav-item rendered only in IsDevelopment().
  • Home.razor — Removed the SectionContent HeaderContent block that was redundantly injecting the title.
  • app.css — Added global .top-nav / .top-nav-item styles to sidestep Blazor CSS isolation scoping between MainLayout and NavMenu.
  • MainLayout.razor.css — Removed padding-bottom: 70px from .app-main; dropped now-unused scoped top-nav rules.
  • NavMenu.razor.css — Removed all bottom-nav styles.
<header class="app-header">
    <div class="header-content">
        <span class="app-title" role="button" tabindex="0" @onclick="..." @onkeydown="...">PoFunQuiz</span>
        <nav class="top-nav">
            <NavLink class="top-nav-item" href="" Match="NavLinkMatch.All">
                <span class="material-icons">home</span>
            </NavLink>
            <NavLink class="top-nav-item" href="leaderboard">
                <span class="material-icons">emoji_events</span>
            </NavLink>
            <NavMenu /> <!-- Diag only, dev mode -->
        </nav>
    </div>
</header>

Screenshot

Top nav with PoFunQuiz title and Home/Leaderboard icons


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…z title in top nav

Co-authored-by: punkouter26 <121304072+punkouter26@users.noreply.github.com>
overflow-y: auto;
padding-bottom: 70px;
padding-bottom: 0;
-webkit-overflow-scrolling: touch;

Choose a reason for hiding this comment

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

The property -webkit-overflow-scrolling: touch; is largely obsolete, as modern browsers (including iOS Safari) now provide smooth scrolling by default. Consider removing this property to simplify the stylesheet unless you specifically need to support legacy iOS versions.

Recommendation:

/* Remove this line unless legacy iOS support is required */

Copilot AI changed the title [WIP] Move home and leaderboard icons to top navigation Move Home/Leaderboard nav icons to top bar; persist PoFunQuiz title in header Mar 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants