fix(#3673): docs site bug sweep - push drawer filters, side menu, multi-sort, responsive grids#3674
Closed
fix(#3673): docs site bug sweep - push drawer filters, side menu, multi-sort, responsive grids#3674
Conversation
…ti-sort, responsive grids Side menu: - Remove stale useGroupShadowDomFixes workaround (chevrons hidden after #3595) - Fix Foundations sub-menu navigation (missing onNavigate handler) Push drawer filters (Examples, Components, Tokens pages): - Replace overlay drawer with push drawer portaled outside content card - Responsive auto-fill grids that reflow when drawer opens - Filter button toggles drawer, modal fallback <1024px with z-index handling - Mobile header z-index drops when drawer open via :has([open]) - Layout padding and --card-padding-h variable for sticky toolbar sync Multi-sort tables: - Enable sort-mode="multi" and _multisort events on all tables - Add sort-order prop for visual 1/2 badges - Remove redundant sort chips, keep filter/search chips only Homepage and misc: - Service pattern links go to /examples/public-form and /examples/workspace - Responsive pattern card grid with auto-fit - Support page button compact, component card thumbnail margin fix - Sticky toolbar constrained to content card width - Mobile header: body-s typography, adjusted spacing and icon padding
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.
Closes #3673
Summary
Hands-on walkthrough of the docs site to find and fix accumulated issues after recent PRs landing (config examples, preview images, app header V2, push drawer, get started pages).
Side menu
useGroupShadowDomFixesshadow DOM workaround. After feat: make icon optional in WorkSideMenuGroup and WorkSideMenuItem #3595 made icons optional, the workaround was hiding the chevron instead of the (now absent) leading icon.onNavigatehandler on Foundations sub-menu.Push drawer filters (Examples, Components, Tokens)
GoabxDrawerwithGoabxPushDraweron all three grid pages--card-padding-hCSS variable keeps sticky toolbar width in sync across breakpoints:has([open])Responsive grids
repeat(4, 1fr)toauto-fillwithminmax(200px, 1fr)auto-fitwithminmax(250px, 1fr)@containerquery instead of@mediaMulti-sort tables
sort-mode="multi"on all threegoa-tableinstances (using feat(#3344): add multi-column sorting to Table and TableSortHeader #3409)_sortto_multisortevent handlingsort-orderattribute for visual "1"/"2" badgesHomepage
/examples/public-forminstead of/examples?userType=citizen/examples/workspaceinstead of/examples?userType=workerSticky toolbar
::beforeconstrained to content card padding (wasleft/right: -9999px)Mobile header
body-swithgreyscale-700(wasmobile-heading-xs)Other
space-xstospace-2xsTest plan