[core] Remove Joy UI code and docs#47939
Conversation
Netlify deploy previewBundle size report
|
github-actions
bot
added
the
PR: out-of-date
Signed-off-by: Marija Najdova <mnajdova@gmail.com>
github-actions
bot
removed
the
PR: out-of-date
When I look at their docs and run with CPU throttling, I see that in their algorithm they render 1 element first and then the rest of the virtual slice (probably for measuring row height?). This is consistent with what we see here, the screenshot is taken before the virtualization had a chance to kick in fully. The change in screenshot count may have changed the performance profile and a race condition kicked in. |
That makes sense, I will disable this test. |
| if (productId === 'joy-ui') { | ||
| return { | ||
| productDisplayName: 'Joy UI', | ||
| IframeWrapper: JoyIframeWrapper, |
There was a problem hiding this comment.
We can probably simplify DemoContext and remove IframeWrapper now.
There was a problem hiding this comment.
I would do it in a follow up to make sure I don't break anything with this PR. Also, @dav-is is already looking into upgrading to the new demo infrastructure, maybe he can check it.
3 participants
Added this entry in the redirecs:
After we have the v8.mui.com we can update it to point to v8 (no difference).
Removed:
packages/mui-joy/*docs/pages/joy-ui/*docs/translations/api-docs-joy/*Removed the @mui/joy dependency from everywhere.
Removed all code related to Joy UI from the scripts, e.g. API generation, handling logic based on Joy UI links etc.
Kept Joy UI in the product popover so people can still access the docs easily. All pages redirects to the v7.mui.com
All of the argos removed screenshots are related to Joy UI.
❓ There is one changed screenshot (docs-components-table/ReactVirtualizedTable.png), that I am not sure where it comes from. Could it be flaky, but somehow surfaced because we have less screenshots now?