Skip to content

Develop - #303

Merged
TOUnail merged 7 commits into
mainfrom
develop
Jun 25, 2026
Merged

TOUnail merged 7 commits into
mainfrom
develop

Conversation

@TOUnail

@TOUnail TOUnail commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Figma Reference: https://www.figma.com/design/W0f6oXULGSX45vwXohMKaq/Footer---Update?node-id=0-1&t=S2Zjl1tabOTmg4DJ-1

Run npm run build and npm run start

Department Footer

Navigate to http://localhost:3000/docs/header-and-navigation/department-footer

Test the following:

  • Unit link
    • Hover, Focus, and Active state
  • Column Navigation
    • Hover, Focus, and Active state
    • Responsive breakpoints
      • Should collapse to 2-columns on mobile

Global Footer

Navigate to http://localhost:3000/docs/header-and-navigation/global-footer

  • Verify that Social Media icons are removed
  • Test Navigation
    • Hover, Focus, and Active state
  • Responsive breakpoints
    • Should collapse on both tablet and mobile

@TOUnail
TOUnail requested review from jkimucla and krcardoza June 23, 2026 16:01
@jkimucla

Copy link
Copy Markdown
Collaborator

@TOUnail I don't see an active or hover state for the unit link. But I do see a focus state.

@jkimucla

Copy link
Copy Markdown
Collaborator

@TOUnail I don't see a active state for the global footer link

@TOUnail

TOUnail commented Jun 25, 2026 •

Copy link
Copy Markdown
Contributor Author

@jkimucla In the Figma, the specs for those states has been changed:

image

I can ask design about the active since it looks like that has been omitted.

@TOUnail

TOUnail commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

@jkimucla I've reverted the colors since there isn't a good determination about link states for blue backgrounds. Active should be fixed now.

@TOUnail
TOUnail merged commit 1e029ac into main Jun 25, 2026
1 check passed
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 2.7.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants