Skip to content

[docs-infra] Fix API prop overflow indicator#4477

Merged
atomiks merged 3 commits intomui:masterfrom
oliviertassinari:fix-overscroll
Apr 14, 2026
Merged

[docs-infra] Fix API prop overflow indicator#4477
atomiks merged 3 commits intomui:masterfrom
oliviertassinari:fix-overscroll

Conversation

@oliviertassinari
Copy link
Copy Markdown
Member

@oliviertassinari oliviertassinari commented Mar 30, 2026

The overflow indicator on the prop got broken at one point. The overflow indicator only works for the API attributes.

Before: https://base-ui.com/react/components/autocomplete#list
SCR-20260330-dfsg

After: https://deploy-preview-4477--base-ui.netlify.app/react/components/autocomplete#list
SCR-20260330-dfrc

I noticed this on #4476.

@oliviertassinari oliviertassinari added type: regression A bug, but worse, it used to behave as expected. scope: docs-infra Involves the docs-infra product (https://www.notion.so/mui-org/b9f676062eb94747b6768209f7751305). labels Mar 30, 2026
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Mar 30, 2026

commit: 81a6113

@oliviertassinari oliviertassinari requested review from aarongarciah and removed request for atomiks, flaviendelangle and michaldudak March 30, 2026 00:45
@mui-bot
Copy link
Copy Markdown

mui-bot commented Mar 30, 2026

Bundle size report

Bundle Parsed size Gzip size
@base-ui/react 0B(0.00%) 0B(0.00%)

Details of bundle changes


Check out the code infra dashboard for more information about this PR.

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 30, 2026

Deploy Preview for base-ui ready!

Name Link
🔨 Latest commit 81a6113
🔍 Latest deploy log https://app.netlify.com/projects/base-ui/deploys/69ddfcbb5ce4310008ffb035
😎 Deploy Preview https://deploy-preview-4477--base-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@atomiks
Copy link
Copy Markdown
Contributor

atomiks commented Mar 30, 2026

I noticed this after the API table was converted into an accordion but wasn't sure if it was intentional 🙈

Can we use this instead so both left/right are faded: https://base-ui.com/react/components/scroll-area#gradient-scroll-fade (unless this component is too expensive to render for each row)

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Apr 9, 2026
@atomiks
Copy link
Copy Markdown
Contributor

atomiks commented Apr 14, 2026

@copilot resolve the merge conflicts in this pull request

# Conflicts:
#	docs/src/components/ReferenceTable/ReferenceTable.css
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Apr 14, 2026
@code-infra-dashboard
Copy link
Copy Markdown

code-infra-dashboard bot commented Apr 14, 2026

Bundle size

Bundle Parsed size Gzip size
@base-ui/react 0B(0.00%) 0B(0.00%)

Details of bundle changes

Performance

Total duration: 1,537.36 ms 🔺+127.71 ms(+9.1%) | Renders: 53 (+0) | Paint: 2,383.44 ms 🔺+198.26 ms(+9.1%)

Test Duration Renders
Tooltip mount (300 contained roots) 97.11 ms 🔺+27.21 ms(+38.9%) 2 (+0)
Popover mount (300 instances) 150.60 ms 🔺+37.75 ms(+33.4%) 2 (+0)
Tabs mount (200 instances) 311.71 ms 🔺+76.31 ms(+32.4%) 4 (+0)
Slider mount (300 instances) 174.68 ms 🔺+38.81 ms(+28.6%) 3 (+0)
Checkbox mount (500 instances) 88.56 ms 🔺+17.79 ms(+25.1%) 1 (+0)

...and 7 more. View full report

Details of benchmark changes


Check out the code infra dashboard for more information about this PR.

@atomiks atomiks merged commit 8bdd9a9 into mui:master Apr 14, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: docs-infra Involves the docs-infra product (https://www.notion.so/mui-org/b9f676062eb94747b6768209f7751305). type: regression A bug, but worse, it used to behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants