-
-
Notifications
You must be signed in to change notification settings - Fork 742
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove Semantic UI from CMSUI #6321
Comments
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Some thoughts about this one: This PLIP has been demoted from Plone 7 roadmap. However, it's still valid, and someone could push for it, although I think that's little sense of doing it. The breakage that this could lead is simply not worth the effort. The community does not have the bandwith to achieve both Plone 7 and this at the same time. Eventually, in the future, an existing Volto site could benefit from the CMSUI in Plone 7, even if it's not Volto based, if all the add-ons that it uses are Plone 6 and Plone 7 compatible. The question would be: Does it makes any sense to leave this open? |
This PLIP is now irrelevant to the Plone 7 roadmap, due to its cautious and time-consuming implementation directives. Also the removal of Semantic UI is a byproduct of the PLIP #6638 and #6322. I think this should be closed. If someone else wants to champion it for Plone 6, then they can resurrect it and take ownership. |
Closing as we decided today. |
PLIP (Plone Improvement Proposal)
Responsible Persons
Proposer: Víctor Fernández de Alba (@sneridagh)
Seconder: Piero Nicolli (@pnicolli)
Abstract
Volto uses since its inception SemanticUI as a component and CSS framework to build PastanagaUI, the Volto current design system. PastanagaUI currently powers both the public part and the CMSUI (the non-public content editing interface).
SemanticUI is unmantained and abandoned since some years ago, which creates a huge technical debt in a critical part of the CMS. Its footprint is also not acceptable for current modern frontend applications leading to big bundles and performance issues.
We will remove Semantic UI from Volto in two chunks, since it's a big effort. We believe that it's more valuable to remove it before from the CMSUI first, then finalize the complete removal of Volto in this PLIP: #6323 By doing this, we could have projects completely SemanticUI free once this PR is completed.
This PLIP is not about implementing Quanta design system in Volto CMSUI. This does not mean that, in the journey to implement this PLIP, we don't adopt some of the styling in Quanta in the new components. In any case, this must not be a blocker for this PLIP to happen.
Implementation directives
This PLIP is targeted to Plone 7. The main goal is to get rid of Semantic UI powered components in the CMSUI.
It is expected to be a long lived branch, and we probably will want some of the changes already available for the API-first story, let's make some assumptions and rules.
All Semantic UI basic, fundamental components should be replaced by their counterparts in
@plone/components
. Every time that we find missing components, we should create them in@plone/components
and use them in Volto components.These components will follow the
@plone/components
best practices, including being written in TypeScript, have accessibility tests, and have a Storybook story.We'd like to have these new components out of this PR and used right away, so let's make all the commits around them as atomic as possible so we can cherry pick them safely and push for them in the main branch.
Every new component used in the CMSUI will be transfered from Volto core to the package
@plone/cmsui
. By doing so, we will keep track of every new "clean" component, and potentially—being able to extract them when we want—we can use them in "real" life right away. It will help also to avoid merging hell.When this PLIP is done, the
src/components/manage
folder should be empty (or almost empty). Code in Volto will remain references and imports to the@plone/cmsui
package.Once we start, only meaningful changes in
main
in CMSUI components will be backported, again to avoid merging hell.Deliverables
@plone/cmsui
package@plone/components
packageThe CMSUI Volto components won’t use SemanticUI and it will be replaced by
@plone/components
components instead.Every new component used in the CMSUI will be transfered from Volto core to the package
@plone/cmsui
.The removal must be compatible with the changes introduced in the new theme and theming story by:
Documentation on:
@plone/components
along with its StoryBook@plone/cmsui
.Risks
This PLIP will break all customizations in CMSUI widgets (including form components), so we will need proper documentation on what we are breaking and how to overcome it.
Eventually, one option would be to create a side add-on that contains the old SemanticUI-based components so people can pull it and force it again. However, it is unlikely that someone would want to go backwards with this change. The CMSUI is something that you barely customize and if so, you will go forwards instead of backwards.
Participants
The text was updated successfully, but these errors were encountered: