Repository navigation
refactor(web): one route-tab strip, row link, sheet header and page title - #1986
Merged
Merged
Maple Review Bot / Maple / review
succeeded
Oct 30, 2026 in 57s
Confidence 10/10 · No issues found
🟢 Confidence 10/10 · safe to merge. Only base-branch files changed since the last review and they are not in this diff; the earlier strip-scrolling and title-wrapping fixes are present at the head.
no findings · tests covered · risk medium
Consolidates four web layout primitives — route-tab strip, row link, sheet header and page title — into the kit and moves ~30 pages onto them. The substitutions are faithful (same classes, attributes and roles) and nothing in the diff is unsafe to merge.
linkTabStripgainsicon,count/countToneandonClick, and every route-tab strip now renders through itrowLinkandROW_LINK_LIFTreplace the four copied stretched-link class stringssheetDetailHeadertakes segmentedkinds and anadornment; peek, review and action sheets use itpageTitle/pageHeroPartsput every page<h1>onpageTitleStyles
What was checked
- Narrow strips still scroll:
underlineTabStripkeepsoverflow-x-autoand tabs stayshrink-0(underline-tabs.ts:27, :39) dashboardPagealready renders thetabsslot inside the sticky area, so investigations'sticky:→tabs:move keeps the strip in place (dashboard-page.ts:180)- No
ROW_STRETCHED_LINK_CLASSreference survives at the head, androwLinkcomposes the identical overlay classes
8fb880c · Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.
Loading