Skip to content

feat(webkit): frame the site footer's column and close it on a full-bleed rule - #1029

Merged
isaque-bock-azion merged 5 commits into
mainfrom
feat/footer-site-frame
Oct 6, 2026
Merged

isaque-bock-azion merged 5 commits into
mainfrom
feat/footer-site-frame

Conversation

@gabriel-lisboa-azion

Copy link
Copy Markdown
Contributor

Summary

  • Footer's main column is a FrameBox (data-testid="layout-footer__frame"): in kind="site" it draws the side rules and all four corner marks; in content it draws none.
  • kind="site" drops the hatched closing band and closes on a full-bleed bottom rule.
  • Spec, story copy, tests, catalog and the footer-root size budget (4 → 3.5 KB, measured 3.15 KB) updated.

How to test

  1. cd packages/webkit && npx vitest run src/components/layout/footer/footer.test.ts → all pass.
  2. Storybook → Components/Layout/Footer → SitePlacement → side rules and a corner mark at each corner, one full-bleed rule under the footer, no hatched band.
  3. Default (content) → no side rules, marks or bottom rule; visually unchanged.
  4. At ≥1536px, SitePlacement → the hatched gutters still flank the framed column.

Notes

  • data-testid="layout-footer__closing" no longer exists; tests that query it will fail. New layout-footer__frame on the column wrapper.
  • kind="site" is visually different: the closing band (64px, 192px from md) is gone, four corner marks appear, the bottom edge is one full-bleed rule. kind="content" only gains the wrapper element.
  • Site-placement visual baselines need regenerating.

# Conflicts:
#	apps/storybook/.storybook/test-visual/__image_snapshots__/linux/components-layout-footer--site-placement--dark-mobile.png
#	apps/storybook/.storybook/test-visual/__image_snapshots__/linux/components-layout-footer--site-placement--dark-tablet.png
#	apps/storybook/.storybook/test-visual/__image_snapshots__/linux/components-layout-footer--site-placement--light-desktop.png
#	apps/storybook/.storybook/test-visual/__image_snapshots__/linux/components-layout-footer--site-placement--light-mobile.png
#	apps/storybook/.storybook/test-visual/__image_snapshots__/linux/components-layout-footer--site-placement--light-tablet.png
@isaque-bock-azion
isaque-bock-azion merged commit b12b255 into main Oct 6, 2026
20 checks passed
@isaque-bock-azion
isaque-bock-azion deleted the feat/footer-site-frame branch October 6, 2026 23:11
@github-actions github-actions Bot mentioned this pull request Oct 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants