Skip to content

24. Sticky nav#4

Open
spiralnebulam31 wants to merge 3 commits intoproductionfrom
dev/24-sticky-nav/app.asana.com/1/100438094841948/project/1204538801343792/task/1213640527696938
Open

24. Sticky nav#4
spiralnebulam31 wants to merge 3 commits intoproductionfrom
dev/24-sticky-nav/app.asana.com/1/100438094841948/project/1204538801343792/task/1213640527696938

Conversation

@spiralnebulam31
Copy link
Copy Markdown

@spiralnebulam31 spiralnebulam31 commented Apr 19, 2026

Implements the “sticky nav” behavior in the starter files by toggling a fixed-nav body class on scroll and applying corresponding fixed-position/animation styles.

Changes:

  • Add .fixed-nav CSS rules to fix the nav, add a shadow, scale the page content, and reveal the logo.
  • Add scroll handler JS to toggle fixed-nav and dynamically add/remove body padding to prevent layout jump.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Implements the “sticky nav” behavior in the starter files by toggling a fixed-nav body class on scroll and applying corresponding fixed-position/animation styles.

Changes:

  • Add .fixed-nav CSS rules to fix the nav, add a shadow, scale the page content, and reveal the logo.
  • Add scroll handler JS to toggle fixed-nav and dynamically add/remove body padding to prevent layout jump.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
24 - Sticky Nav/style-START.css Adds .fixed-nav styles for fixed positioning, shadow, content scaling, and logo reveal via max-width transition.
24 - Sticky Nav/index-START.html Adds scroll-based logic to apply fixed-nav and adjust body padding based on nav height.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread 24 - Sticky Nav/index-START.html Outdated
Comment thread 24 - Sticky Nav/index-START.html
Comment thread 24 - Sticky Nav/style-START.css Outdated
@spiralnebulam31 spiralnebulam31 marked this pull request as ready for review April 19, 2026 15:50
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