Skip to content

fix(web): align consolidated header logo, version, and nav on mobile - #2054

Merged
elibosley merged 1 commit into
mainfrom
fix/consolidated-header-mobile-padding
Jul 24, 2026
Merged

fix(web): align consolidated header logo, version, and nav on mobile#2054
elibosley merged 1 commit into
mainfrom
fix/consolidated-header-mobile-padding

Conversation

@elibosley

@elibosley elibosley commented Jul 24, 2026

Copy link
Copy Markdown
Member

What

Follow-up mobile/desktop polish for the consolidated `` (shipped in #2037).

  • Keep the logo and account controls (name / bell / menu / avatar) centered on the same row so they line up on mobile, instead of stretching the logo block and pinning the logo to the top (which detached it from the nav cluster).
  • Let the version tag flow directly under the logo — sharing its left edge — with a small `position: relative` offset so its baseline lines up with the bottom of the account controls, at both mobile and desktop breakpoints.

Why

The prior mobile layout floated the version out of flow and stretched the logo block, causing the logo/hamburger to misalign and the version to sit misaligned against the logo and cramped against the bottom nav divider.

Testing

Deployed to devgen.local (Unraid 7.3.2) and verified logo/nav alignment and version placement on mobile and desktop viewports.

Refs OS-620

Summary by CodeRabbit

  • Style
    • Improved mobile header layout and alignment.
    • Adjusted version text positioning to display consistently beneath the logo and align with account controls.

Mobile header polish for the consolidated <unraid-header>:
- Keep the logo and account controls (name/bell/menu/avatar) centered on
  the same row so they line up, instead of stretching the logo block and
  pinning the logo to the top.
- Let the version tag flow directly under the logo (shared left edge) with
  a small relative offset so its baseline lines up with the bottom of the
  account controls, at both mobile and desktop breakpoints.

Refs OS-620.
@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 5a047e8b-ed1a-4661-9ce7-77f2eef8a9fc

📥 Commits

Reviewing files that changed from the base of the PR and between 1e7ae56 and 1389122.

📒 Files selected for processing (1)
  • web/src/components/Header.standalone.vue

Walkthrough

Updated the standalone header’s responsive CSS so the version remains in normal flow beneath the logo and is visually lowered to align with account controls.

Changes

Header layout

Layer / File(s) Summary
Version alignment styling
web/src/components/Header.standalone.vue
The mobile layout comments describe the shared grid row and flowing version placement; .uh-version now uses a relative 0.5rem downward offset across breakpoints.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • unraid/api#2037: Introduced the consolidated header mobile layout refined by these positioning changes.

Poem

A bunny nudges version text down low,
Beneath the logo’s tidy glow.
The controls align, the rows now flow,
A softer header steals the show. 🐇

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: mobile alignment of the header logo, version tag, and nav controls.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/consolidated-header-mobile-padding

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Jul 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 53.07%. Comparing base (1e7ae56) to head (1389122).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2054      +/-   ##
==========================================
- Coverage   53.07%   53.07%   -0.01%     
==========================================
  Files        1041     1041              
  Lines       72425    72425              
  Branches     8354     8349       -5     
==========================================
- Hits        38440    38439       -1     
- Misses      33858    33859       +1     
  Partials      127      127              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

Copy link
Copy Markdown
Contributor

This plugin has been deployed to Cloudflare R2 and is available for testing.
Download it at this URL:

https://preview.dl.unraid.net/unraid-api/tag/PR2054/dynamix.unraid.net.plg

@elibosley
elibosley merged commit 12c23d8 into main Jul 24, 2026
13 of 14 checks passed
@elibosley
elibosley deleted the fix/consolidated-header-mobile-padding branch July 24, 2026 15:46
@github-actions

Copy link
Copy Markdown
Contributor

🔄 PR Merged - Plugin Redirected to Staging

This PR has been merged and the preview plugin has been updated to redirect to the staging version.

For users testing this PR:

  • Your plugin will automatically update to the staging version on the next update check
  • The staging version includes all merged changes from this PR
  • No manual intervention required

Staging URL:

https://preview.dl.unraid.net/unraid-api/dynamix.unraid.net.plg

Thank you for testing! 🚀

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.

1 participant