- Issue created by @pdureau
- First commit to issue fork.
- Assigned to jldust
- Status changed to Needs review
10 months ago 6:54pm 19 January 2024 - πΊπΈUnited States jldust
So glad to hear you are using Prototype for your testing and thank you for your detailed feedback, I appreciate it! I've opened a MR with updates based on the feedback, if you could review it that would be great.
- Merge request !94Issue 3413130 - Updating Teaser component and the breadcrumb component β (Merged) created by jldust
- Assigned to pdureau
- Issue was unassigned.
- Status changed to Needs work
10 months ago 3:04pm 21 January 2024 - π«π·France pdureau Paris
breadcrumb
It seems my feedback about breadrcumb presenter template was not clear enough.
I was not suggesting to add the separator prop in templates/page/page.html.twig but to be sure breadcrumb.html.twig is sending the expected data to breadcrumb component.Empty components
Some components have no props and no slots:
I am not sure SDC is loading slots without a definition:
slot: content: {} link: {}
Also, 2 components have placeholders props "due to a core issue":
- menu
- menu--navigation
- πΊπΈUnited States jldust
jldust β changed the visibility of the branch 3413130-sdc-components-fix to hidden.
- πΊπΈUnited States jldust
jldust β changed the visibility of the branch 3413130-sdc-components-fix to active.
- Status changed to Needs review
10 months ago 6:05pm 24 January 2024 - πΊπΈUnited States jldust
I realized that I miss read your feedback and have updated the component accordingly, let me know if this works!
- πΊπΈUnited States jldust
Also we are aware of the empty props and are currently working to update those components, the original core issue is here: https://www.drupal.org/project/drupal/issues/3385283 π Components without props throw error Fixed
-
jldust β
committed 57fcc323 on 5.x
Issue #3413130 by jldust, pdureau: SDC components: Fix JSON schema usage
-
jldust β
committed 57fcc323 on 5.x
- Status changed to RTBC
10 months ago 6:10pm 31 January 2024 - πΊπΈUnited States jldust
We have pulled in these changes to the main 5.x branch, these updates will be included in the next tagged release.
- Status changed to Fixed
10 months ago 12:10am 2 February 2024 - Status changed to Fixed
10 months ago 1:19pm 6 February 2024