- Issue created by @mandclu
- @mandclu opened merge request.
- Status changed to Needs review
almost 2 years ago 8:21am 9 February 2023 - 🇨🇦Canada mandclu
A first attempt at the proposed changes is now in the merge request.
- Status changed to Fixed
almost 2 years ago 11:11am 9 February 2023 - 🇨🇦Canada mandclu
I tested this on a fresh install of Acquia CMS and it worked as intended, including the preview thumbnails for the components.
I did get a fatal error when saving a media image with an SVG file attached, but that turned out to be a problem with focal_point, and a fix is available in 🐛 FocalPointManager::saveCropEntity(): Argument #3 must be of type int, null given Fixed .
In case anyone is searching in this issue queue for the relevant error, here it is:
TypeError: Drupal\focal_point\FocalPointManager::saveCropEntity(): Argument #3 ($width) must be of type int, null given, called in /app/web/modules/contrib/focal_point/focal_point.module on line 99 in Drupal\focal_point\FocalPointManager->saveCropEntity() (line 94 of /app/web/modules/contrib/focal_point/src/FocalPointManager.php) Automatically closed - issue fixed for 2 weeks with no activity.