Skip to content

feat: add footer visibility toggle with localStorage persistence#548

Open
NeerajPise wants to merge 7 commits intotenstorrent:devfrom
NeerajPise:dev-neeraj/493-feat-footer-toggle-button
Open

feat: add footer visibility toggle with localStorage persistence#548
NeerajPise wants to merge 7 commits intotenstorrent:devfrom
NeerajPise:dev-neeraj/493-feat-footer-toggle-button

Conversation

@NeerajPise
Copy link
Contributor

@NeerajPise NeerajPise commented Nov 21, 2025

This PR implements a user-controlled footer visibility toggle system, allowing users to show/hide the footer. The visibility state persists across page reloads using localStorage.
This solves #493

Functional testing -

  • Footer toggles on/off when clicking chevron buttons
  • Footer state persists after page reload
  • Footer state persists across navigation between routes
  • Default state: visible on homepage, respects localStorage on other pages
  • Toggle works in both loading and loaded footer states

Theme Testing -

  • Buttons visible in dark mode
  • Buttons visible in light mode

Visual Testing -

  • Up button visible and centered when footer is hidden
  • Down button visible and centered when footer is shown
  • Buttons are bold and easily clickable
  • Smooth animations on show/hide transitions

Demo video -

Screen.Recording.2025-11-20.at.9.35.00.PM.mp4

@anirudTT anirudTT added the community Community-driven issues and suggestions. label Jan 8, 2026
@anirudTT
Copy link
Collaborator

@NeerajPise : could you resolve the conflicts , and I can get started reviewing this PR

@anirudTT anirudTT requested review from anirudTT and Copilot and removed request for anirudTT January 12, 2026 23:14
@anirudTT anirudTT self-assigned this Jan 12, 2026
Copy link
Contributor

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

This PR adds a user-controlled footer visibility toggle feature with localStorage persistence, enabling users to show or hide the footer component. The visibility state is remembered across page reloads and navigation. The implementation replaces the previous deployment-based footer visibility logic with a more flexible user-controlled approach.

Changes:

  • Introduced a new FooterVisibilityContext with provider and custom hook for state management
  • Added chevron toggle buttons (up/down) to the Footer component for showing/hiding functionality
  • Removed the deployment-based footer conditional rendering from MainLayout
  • Integrated the FooterVisibilityProvider into the App component hierarchy
  • Applied formatting fixes (removed trailing spaces in bug report template, changed let to const)

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
app/frontend/src/contexts/FooterVisibilityContext.ts Defines the FooterVisibilityContext type and creates the context
app/frontend/src/providers/FooterVisibilityContext.tsx Implements the provider with localStorage persistence and default visibility logic
app/frontend/src/hooks/useFooterVisibility.ts Custom hook for consuming the FooterVisibilityContext with error handling
app/frontend/src/App.tsx Wraps the app in FooterVisibilityProvider to enable context throughout the app
app/frontend/src/layouts/MainLayout.tsx Removes deployment-based footer hiding logic, always renders Footer component
app/frontend/src/components/Footer.tsx Adds toggle buttons, AnimatePresence wrapper, and visibility state handling

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

@NeerajPise
Copy link
Contributor Author

@anirudTT I'll resolve the conflicts and update the PR after checking Copilot's suggested fixes soon

@NeerajPise NeerajPise force-pushed the dev-neeraj/493-feat-footer-toggle-button branch from 01af141 to 5f9592f Compare January 13, 2026 23:20
@NeerajPise NeerajPise force-pushed the dev-neeraj/493-feat-footer-toggle-button branch from 5f9592f to 699939f Compare January 13, 2026 23:26
@NeerajPise
Copy link
Contributor Author

@anirudTT Fixed the conflicts and integrated Copilot's suggestions after a review

Copy link
Contributor

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

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


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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Community-driven issues and suggestions.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants