- Issue created by @texas_tater
- 🇺🇸United States dalemoore
Same question. I just now noticed that new button and was confused where it came from. I thought maybe it was somehow sneaking in what this issue does: https://www.drupal.org/project/drupal/issues/3338369 ✨ Layout Builder - Add a refresh layout defaults feature Needs review
- 🇬🇧United Kingdom dunx
https://www.drupal.org/project/drupal/issues/3338369 ✨ Layout Builder - Add a refresh layout defaults feature Needs review was from us over at Cyber-Duck and we're already using the patch successfully on projects. It's completely different functionality to what that new button suggests it's trying to do.
So, we don't know what this new button does either, except that clicking on it appears to do nothing in our use case. It's also a link rather than a button, so doesn't match the UI for the other items in that space as per OP screenshots.
We've only just replaced LBM with LBIM, so not 100% familiar with the differences.
- 🇺🇸United States bkosborne New Jersey, USA
Looks like the link is important and is used to rebuild the layout after changes were made to the layout. It's clicked programmatically via iframe.js. So I don't think it needs to be visible.
- @bkosborne opened merge request.
- Status changed to Needs review
over 1 year ago 6:33pm 15 June 2023 - 🇺🇸United States bkosborne New Jersey, USA
Created an MR to hide it visually and for screen readers. The JS can still access it and use it.
- Status changed to RTBC
over 1 year ago 4:27pm 16 June 2023 - 🇺🇸United States mark_fullmer Tucson
I agree with the analysis that the link serves a purpose but doesn't need to be visible. This checks out. Marking as RTBC!
- 🇺🇸United States pfrilling Minster, OH
Uploading a patch for use with composer.