- Issue created by @juagarc4
- last update
about 1 year ago 29,482 pass - @juagarc4 opened merge request.
- Status changed to Needs review
about 1 year ago 7:21pm 22 September 2023 - last update
about 1 year ago 29,482 pass - πΊπΈUnited States smustgrave
Think thereβs some BC to consider. This would affect other sites and maybe not in a way they wanted.
- Status changed to Needs work
about 1 year ago 4:37pm 24 September 2023 - πΊπΈUnited States smustgrave
Yea this will need work as the current layout says 33/34/33 but that wouldn't be true. If that layout is updated may need an upgrade path too
- π©πͺGermany juagarc4
Hi @smustgrave,
Thanks for your comments. I'm totally agree with the upgrade path.
As far I've seen, we need to update two modules: layout_builder and layout_discovery and the themes which are already using this class name: Olivero and Stable9.
Maybe we can start adding the new class together with the current one and adding a notice in the code indicating that the current class name will be removed in the next major version. It can reduce the BC issues and let the developers to adapt the code to the new class name.Should we create a ticket for each case or can we address all changes in this one?
I would tend to create a ticket for each case related to this one as a main ticket. What do you mean? - π§πͺBelgium michel.g
We should extend this to the two column region with 67% and 33% width