-
-
Notifications
You must be signed in to change notification settings - Fork 615
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
fix: Listing block default summary should use ul tag for its listing #6318
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for plone-components canceled.
|
@plone/volto-accessibility we need to have a look at this change and provide feedback |
@JeffersonBledsoe never mind the review request there needs to be some changes done to this pull request, one of them being not reintroducing Semantic-ui component within the Summary Template |
@ana-oprea there are a couple of issues currently:
|
@ana-oprea Thanks for removing SemanticUI fromt here! However, I won't remove the current |
This PR continues the work from PR #4994, which is now outdated. To ensure progress on the development, I've opened this new PR to carry the changes forward and address any remaining tasks or updates.