- Issue created by @codebymikey
- Merge request !4Issue #3476370: Improve the Bulk alt text form behaviour → (Open) created by codebymikey
codebymikey → changed the visibility of the branch 3476370-use-configured-image-style to hidden.
Update the code a bit to meet best practices. There's still some other improvements to be made such as references to "alt tag", should probably be "alt text", but that should be handled in a separate issue with an update hook after this issue is resolved.
- Assigned to codebymikey
- Status changed to Needs review
5 months ago 5:06am 15 December 2024 I had this exact issue. I did not have the Large image style, and experienced the error below, along with a white screen when going to admin/config/media/ai_image_bulk_alt_text. Thanks to this post, I was able to add the Large image style and the page worked correctly. Thanks for the great module! My error is below:
Message Error: Call to a member function transformDimensions() on null in template_preprocess_image_style() (line 288 of /var/www/html/website/web/core/modules/image/image.module).
Severity Error- Issue was unassigned.
- 🇮🇳India prashant.c Dharamshala
- Overall changes looks good
- Need to fix the warning
PHP Deprecated: Using ${var} in strings is deprecated, use {$var} instead in /var/www/html/modules/contrib/ai_image_alt_text/modules/ai_image_bulk_alt_text/src/Form/BulkAltTextGenerateForm.php on line 304 - Added some comments to the PR
- Unassigning from @codebymikey as there is no activity since September.
- Keeping in the NR