Skip to content

fix(web): polish combined header on mobile and desktop - #2109

Merged
Eli Bosley (elibosley) merged 3 commits into
mainfrom
codex/mobile-combined-header
Oct 10, 2026
Merged

Eli Bosley (elibosley) merged 3 commits into
mainfrom
codex/mobile-combined-header

fix(web): address combined header review feedback

bfaae1c
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Oct 10, 2026 in 0s

53.31% (-0.16%) compared to 64eac12

View this Pull Request on Codecov

53.31% (-0.16%) compared to 64eac12

Details

Codecov Report

❌ Patch coverage is 21.37405% with 206 lines in your changes missing coverage. Please review.
✅ Project coverage is 53.31%. Comparing base (64eac12) to head (bfaae1c).

Files with missing lines Patch % Lines
web/test-pages/combined-header.js 0.00% 197 Missing and 1 partial ⚠️
web/src/components/Header/HeaderVersion.vue 27.27% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2109      +/-   ##
==========================================
- Coverage   53.47%   53.31%   -0.16%     
==========================================
  Files        1044     1045       +1     
  Lines       72706    72906     +200     
  Branches     8416     8457      +41     
==========================================
- Hits        38876    38872       -4     
- Misses      33703    33906     +203     
- Partials      127      128       +1     

☔ 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.