feat(footer): add Documentation and Contact support links#175
Open
nunoeufrasio wants to merge 2 commits into
Open
feat(footer): add Documentation and Contact support links#175nunoeufrasio wants to merge 2 commits into
nunoeufrasio wants to merge 2 commits into
Conversation
Add two links to the footer Platform section: - Documentation: opens docs.linuxfoundation.org/lfx/crowdfunding in a new tab - Contact support: opens the Intercom messenger via useIntercom().show(), mirroring the For Companies Contact our team button Also add Documentation to the navbar More menu (below About) with external new-tab handling, and left-align footer action buttons. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Signed-off-by: Nuno Eufrasio <nmeufrasio@gmail.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the frontend navigation to add a Documentation link (opening the Linux Foundation docs in a new tab) and a Contact support footer entry that opens Intercom, aligning footer behavior with the existing Intercom CTA introduced in #174.
Changes:
- Added a Documentation entry to the header “More” menu with an
externalflag to rendertarget="_blank"+rel="noopener noreferrer". - Added Documentation and Contact support entries to the footer “Platform” section, with Contact support opening Intercom via
useIntercom().show(). - Extended footer link typing to support a declarative
intercom: truelink variant.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/app/types/footer.types.ts | Extends footer menu link union type to support an intercom: true variant. |
| frontend/app/config/menu/header.ts | Adds “Documentation” child item and introduces optional external flag on header children. |
| frontend/app/config/menu/footer.ts | Adds “Documentation” (external) and “Contact support” (Intercom) entries to footer Platform links. |
| frontend/app/components/shared/layout/footer.vue | Renders Intercom footer link as a button and wires it to useIntercom().show(); aligns button text styling. |
| frontend/app/components/shared/layout/components/mobile-menu.vue | Applies target/rel for external “More” children links in the mobile menu. |
| frontend/app/components/shared/layout/components/desktop-nav.vue | Applies target/rel for external “More” children links in the desktop dropdown. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
mlehotskylf
reviewed
Jun 16, 2026
mlehotskylf
reviewed
Jun 16, 2026
mlehotskylf
requested changes
Jun 16, 2026
Address review comments from @mlehotskylf: - config/menu/footer.ts: point Documentation to the in-app /docs route instead of the deprecated docs.linuxfoundation.org CF docs - config/menu/header.ts: point More-menu Documentation to /docs and drop the now-unused external flag - layout/components/desktop-nav.vue, mobile-menu.vue: revert external new-tab handling (no longer needed for an in-app link) The in-app docs system is introduced in #166 (AppRoute.Docs = '/docs'). Resolves 2 review threads. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Signed-off-by: Nuno Eufrasio <nmeufrasio@gmail.com>
Contributor
Author
Review Feedback AddressedCommit: e8f89d4 Changes Made
Both Documentation links are now internal links to the in-app docs system introduced in #166. "Contact support" (Intercom) is unchanged. Threads Resolved2 of 2 unresolved threads addressed in this iteration. |
mlehotskylf
approved these changes
Jun 16, 2026
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.
What
Adds two links to the footer Platform section, and a Documentation entry to the navbar More menu:
useIntercom().show()mechanism as the "Contact our team" button from fix(for-companies): replace broken /contact links with Intercom #174.How
useIntercom()is resolved infooter.vue'ssetup()with a localopenIntercom()handler. The menu config stays declarative via a newintercom: truelink variant.externalflag so the Documentation link renders withtarget="_blank"+rel="noopener noreferrer"(desktop nav and mobile menu).text-left) so they line up with the anchor links — also fixes pre-existing drift on "Start a Fundraise".Related
Checklist
show🤖 Generated with Claude Code