Skip to content

Cleaned React.Common exposed react components that were marked for v10 removal#6684

Merged
bdukes merged 1 commit intodnnsoftware:developfrom
valadas:clean-v10-react-common
Sep 5, 2025
Merged

Cleaned React.Common exposed react components that were marked for v10 removal#6684
bdukes merged 1 commit intodnnsoftware:developfrom
valadas:clean-v10-react-common

Conversation

@valadas
Copy link
Copy Markdown
Contributor

@valadas valadas commented Sep 5, 2025

5 years ago Matt and me (and few others) did a massive cleanup of the react common components after the react upgrade debacle. At that time we marked that some components should no longer be exposed in the bundle directly. This was missed when we created v10.0.0

This PR removes those. The bundle size reduced from 2303kB to 1647Kb (compressed from 754kB to 354kB).

I have visited each persona bar module and clicked around different tabs and pickers, etc. and did not notice anything broken.

…0 removal

5 years ago Matt and me (and few others) did a massive cleanup of the react common components after the react upgrade debacle. At that time we marked that some components should no longer be exposed in the bundle directly. This was missed when we created v10.0.0

This PR removes those. The bundle size reduced from 2303kB to 1647Kb (compressed from 754kB to 354kB).

I have visited each persona bar module and clicked around different tabs and pickers, etc. and did not notice anything broken.
@valadas valadas added this to the 10.1.1 milestone Sep 5, 2025
@bdukes bdukes enabled auto-merge (rebase) September 5, 2025 19:18
@bdukes bdukes merged commit 810744e into dnnsoftware:develop Sep 5, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants