Add native bottom navigation mock - #10
Open
frankieroberto wants to merge 3 commits into
Open
Conversation
frankieroberto
temporarily deployed
to
nhs-app-prot-add-native-sgvwzw
August 27, 2026 15:58
Inactive
There was a problem hiding this comment.
Pull request overview
Adds an alternative “native” bottom navigation mock to the prototype kit, switching the layout footer between the existing web footer and the new native-style bottom navigation based on data.web.
Changes:
- Replace the inline footer macro call with a conditional include that selects either the web footer or native bottom navigation.
- Add new include partials for the web footer and native bottom navigation.
- Add unread-message state helper and corresponding Sass for the native bottom navigation.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| app/views/layout.html | Switches footer rendering to a conditional include (web footer vs native bottom navigation). |
| app/views/_includes/web-footer.html | Extracts the existing footer macro call into a reusable include. |
| app/views/_includes/native-bottom-navigation.html | Adds the native-style bottom navigation markup and unread icon state logic. |
| app/locals.js | Introduces a view helper to determine whether unread messages exist. |
| app/assets/sass/native-bottom-navigation.scss | Adds styles for the fixed native bottom navigation and page bottom padding. |
| app/assets/sass/main.scss | Imports the new native bottom navigation styles. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
frankieroberto
temporarily deployed
to
nhs-app-prot-add-native-sgvwzw
August 27, 2026 16:28
Inactive
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This adds a mockup of the native bottom-navigation component, as an alternative to the web version