- Issue created by @mably
- Merge request !28Issue #3420037: have SvgImageWidget inherit from ImageWidget instead of FileWidget β (Open) created by mably
- Status changed to Needs review
5 months ago 3:04pm 13 February 2024 - π§πͺBelgium herved
Coming from π Replace upload validators with new ones introduced in Drupal 10.2 Needs work as we are having duplicate extensions descriptions as well on our project.
This issue makes more sense to me and looks like a much cleaner way to add SVG support.
Tested on core 10.2, works perfectly, thank you.Only 2 minor nits:
1. There is a typo in comments: Druapal > Drupal
2. I think the plugin annotation shouldn't be there assvg_image_field_formatter_info_alter()
already registers the class. But this is probably not directly related to this issue. - πΊπΈUnited States mortona2k Seattle
I applied the latest commit, but I still see the duplicate allowed types. Would the ticket in #8 need to be rebuilt on this one to fix that?
Anything I can help with?
- π«π·France mably
Allowed types are not displayed twice when using this issue MR:
- πΊπΈUnited States mortona2k Seattle
You're right, I must have had my wires crossed in testing.
The change looks good to me, ripping out a bunch of stuff that duplicates what ImageWidget does.
Looks like you handled the validators from #3413668.
I think it's RBTC, just looking for confirmation.
- π§π·Brazil Lucas Fernandes
Hey everyone,
I was having the same issue and this fix it worked for me as well.
Thanks for this.
- Status changed to RTBC
3 months ago 9:16pm 11 April 2024 - π§π·Brazil murilohp
That's great, here's a static patch to be applied on composer