Skip to content

Fix mobile navbar: remove inline styles and fix navigation IDs #365

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
aurelianware merged 6 commits into from
Feb 12, 2026
Merged

Fix mobile navbar: remove inline styles and fix navigation IDs #365

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
3d86b40
Initial plan
Copilot Feb 10, 2026
32473cf
Fix mobile navbar display issue - remove inline display:flex styles
Copilot Feb 10, 2026
bce45a5
Merge branch 'main' into copilot/fix-mobile-navbar-display
aurelianware Feb 12, 2026
d9a5a73
Remove redundant inline styles and fix platform.html nav ID
Copilot Feb 12, 2026
cf6706b
Update assessment.html to sync with markdown changes from main
Copilot Feb 12, 2026
47d3ecd
Fix YAML linting errors - remove trailing spaces in test-dotnet.yml
Copilot Feb 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions site/css/sentinel.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -307,6 +307,7 @@ nav ul {
align-items: center;
flex-wrap: wrap;
gap: var(--spacing-md);
padding: 0;
}

nav li {
Expand Down
2 changes: 1 addition & 1 deletion site/index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@
<button class="mobile-menu-toggle" aria-label="Toggle navigation menu" aria-expanded="false" id="mobileMenuToggle">
☰
</button>
<ul style="list-style:none; padding:0; display:flex; justify-content:center; gap:30px; flex-wrap:wrap;" id="mainNav">
<ul style="list-style:none; padding:0;" id="mainNav">
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

The inline styles list-style:none; padding:0; are now redundant since the CSS rule nav ul (sentinel.css:303-311) already defines both list-style: none and padding: 0. These inline styles can be completely removed from all ul elements in the nav.

Suggested change
<ul style="list-style:none; padding:0;" id="mainNav">
<ul id="mainNav">

Copilot uses AI. Check for mistakes.
<li><a href="index.html" style="color:#00ffff;" aria-current="page">Home</a></li>
<li><a href="solutions-payers.html" style="color:#888;">Solutions</a></li>
<li><a href="pricing.html" style="color:#888;">Pricing</a></li>
Expand Down
2 changes: 1 addition & 1 deletion site/login.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@
<a href="#main-content" class="skip-to-main">Skip to main content</a>

<nav style="margin-bottom:20px;">
<ul style="list-style:none; padding:0; display:flex; justify-content:center; gap:30px; flex-wrap:wrap;">
<ul style="list-style:none; padding:0;">
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

The inline styles list-style:none; padding:0; are now redundant since the CSS rule nav ul (sentinel.css:303-311) already defines both list-style: none and padding: 0. These inline styles can be completely removed from all ul elements in the nav.

Suggested change
<ul style="list-style:none; padding:0;">
<ul>

Copilot uses AI. Check for mistakes.
<li><a href="index.html" style="color:#888;">Home</a></li>
<li><a href="solutions-payers.html" style="color:#888;">For Payers</a></li>
<li><a href="solutions-providers.html" style="color:#888;">For Providers</a></li>
Expand Down
2 changes: 1 addition & 1 deletion site/platform.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@
<button class="mobile-menu-toggle" aria-label="Toggle navigation menu" aria-expanded="false" id="mobileMenuToggle">
☰
</button>
<ul style="list-style:none; padding:0; display:flex; justify-content:center; gap:30px; flex-wrap:wrap;" id="platformNav">
<ul style="list-style:none; padding:0;" id="platformNav">
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

The mobile navigation JavaScript (mobile-nav.js) looks for an element with id="mainNav", but this page uses id="platformNav". This means the mobile menu toggle button will not function on this page. Either rename this ID to "mainNav" to match the other pages, or update mobile-nav.js to handle multiple nav IDs.

Copilot uses AI. Check for mistakes.
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

The inline styles list-style:none; padding:0; are now redundant since the CSS rule nav ul (sentinel.css:303-311) already defines both list-style: none and padding: 0. These inline styles can be completely removed from all ul elements in the nav.

Suggested change
<ul style="list-style:none; padding:0;" id="platformNav">
<ul id="platformNav">

Copilot uses AI. Check for mistakes.
<li><a href="index.html" style="color:#888;">Home</a></li>
<li><a href="solutions-payers.html" style="color:#888;">Solutions</a></li>
<li><a href="pricing.html" style="color:#888;">Pricing</a></li>
Expand Down
2 changes: 1 addition & 1 deletion site/pricing.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
<button class="mobile-menu-toggle" aria-label="Toggle navigation menu" aria-expanded="false" id="mobileMenuToggle">
☰
</button>
<ul style="list-style:none; padding:0; display:flex; justify-content:center; gap:30px; flex-wrap:wrap;" id="mainNav">
<ul style="list-style:none; padding:0;" id="mainNav">
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

The inline styles list-style:none; padding:0; are now redundant since the CSS rule nav ul (sentinel.css:303-311) already defines both list-style: none and padding: 0. These inline styles can be completely removed from all ul elements in the nav.

Suggested change
<ul style="list-style:none; padding:0;" id="mainNav">
<ul id="mainNav">

Copilot uses AI. Check for mistakes.
<li><a href="index.html" style="color:#888;">Home</a></li>
<li><a href="solutions-payers.html" style="color:#888;">Solutions</a></li>
<li><a href="pricing.html" style="color:#00ffff;" aria-current="page">Pricing</a></li>
Expand Down
2 changes: 1 addition & 1 deletion site/solutions-payers.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
<button class="mobile-menu-toggle" aria-label="Toggle navigation menu" aria-expanded="false" id="mobileMenuToggle">
☰
</button>
<ul style="list-style:none; padding:0; display:flex; justify-content:center; gap:30px; flex-wrap:wrap;" id="mainNav">
<ul style="list-style:none; padding:0;" id="mainNav">
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

The inline styles list-style:none; padding:0; are now redundant since the CSS rule nav ul (sentinel.css:303-311) already defines both list-style: none and padding: 0. These inline styles can be completely removed from all ul elements in the nav.

Suggested change
<ul style="list-style:none; padding:0;" id="mainNav">
<ul id="mainNav">

Copilot uses AI. Check for mistakes.
<li><a href="index.html" style="color:#888;">Home</a></li>
<li><a href="solutions-payers.html" style="color:#00ffff;" aria-current="page">Solutions</a></li>
<li><a href="pricing.html" style="color:#888;">Pricing</a></li>
Expand Down
2 changes: 1 addition & 1 deletion site/solutions-providers.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
<button class="mobile-menu-toggle" aria-label="Toggle navigation menu" aria-expanded="false" id="mobileMenuToggle">
☰
</button>
<ul style="list-style:none; padding:0; display:flex; justify-content:center; gap:30px; flex-wrap:wrap;" id="mainNav">
<ul style="list-style:none; padding:0;" id="mainNav">
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

The inline styles list-style:none; padding:0; are now redundant since the CSS rule nav ul (sentinel.css:303-311) already defines both list-style: none and padding: 0. These inline styles can be completely removed from all ul elements in the nav.

Suggested change
<ul style="list-style:none; padding:0;" id="mainNav">
<ul id="mainNav">

Copilot uses AI. Check for mistakes.
<li><a href="index.html" style="color:#888;">Home</a></li>
<li><a href="solutions-payers.html" style="color:#888;">Solutions</a></li>
<li><a href="pricing.html" style="color:#888;">Pricing</a></li>
Expand Down
Loading