- Issue created by @technoveltyco
- @technoveltyco opened merge request.
- @technoveltyco opened merge request.
- Merge request !3368Draft: Issue #3338369: Layout Builder - Add a refresh layout defaults feature β (Open) created by technoveltyco
- π¬π§United Kingdom technoveltyco
Proposed patch
i created the patches for the corresponding Drupal Core versions 9.5.x, 10.0.x and 10.1.x
- Patch for Drupal Core 9.5.x: 3338369-3-9.5.x--Layout-Builder-Add-a-refresh-layout-feature.patch β
- Patch for Drupal Core 10.0.x: 3338369-3-10.0.x--Layout-Builder-Add-a-refresh-layout-feature.patch β
- Patch for Drupal Core 10.1.x: 3338369-3-10.1.x--Layout-Builder-Add-a-refresh-layout-feature.patch β
Likewise, there are opened merge request with the following branches for each of the Drupal Core versions:- Merge request for Drupal 9.5.x: https://git.drupalcode.org/project/drupal/-/merge_requests/3366
- Merge request for Drupal 10.0.x: https://git.drupalcode.org/project/drupal/-/merge_requests/3367
- Merge request for Drupal 10.1.x: https://git.drupalcode.org/project/drupal/-/merge_requests/3368
All these patches include the same implementation of the Refresh layout defaults button as described in the solution proposal section.Bear in mind that the automated tests are still pending to be included, but please feel free to test, validate and give feedback for the implementation of this new feature, so the remaining tasks can be work out later on as the discussion follows up.
Many thanks!
- Status changed to Needs review
almost 2 years ago 10:40am 2 February 2023 The last submitted patch, 6: 3338369-3-10.0.x--Layout-Builder-Add-a-refresh-layout-feature.patch, failed testing. View results β
The last submitted patch, 6: 3338369-3-10.1.x--Layout-Builder-Add-a-refresh-layout-feature.patch, failed testing. View results β
- Status changed to Postponed
9 months ago 3:56am 8 March 2024 - π³πΏNew Zealand danielveza Brisbane, AU
Super cool that some initial work has been done here. It would be awesome if we could find a clean way to resolve this.
However I don't think this is very safe at the moment as it relies on section labels which I don't think we should do
For example "Section 2" on a default Layout may not be the same as "Section 2" on an overriden template. I think we might need need to postpone this on β¨ Add UUID to sections Needs work
It could be nice to have some configuration on the form. Something like
- Only add new blocks to matching Section
- Only add new sections
- Complete refresh
- π¬π§United Kingdom Alina Basarabeanu
A new patch for the Drupal core 10.3.6