Ability to use original svg dimensions for height/width when used as image

Please fix trailing spaces.

What impact will this patch have on existing fields that have not been configured this way? If there is any chance this could cause regressions for existing users, then the approach will need to be implemented with a new optional checkbox, as mentioned in the issue summary.

+++ b/web/modules/contrib/svg_image_field/src/Plugin/Field/FieldFormatter/SvgImageFieldFormatter.php
@@ -258,6 +258,32 @@ public function viewElements(FieldItemListInterface $items, $langcode) {
+          $attributes = ['width'=>$width, 'height'=>$height];

Might make sense to cast to an int (and optionally be sanitized) in one single place, before being sent to the Attributes array.

Vadym.Tseiko → made their first commit to this issue’s fork.

Re-roll patch #2. Fix phpcs.

Code changes are okay, but marking NW to think through and address the second question on #3.

Hi @jwilson3, I thought that in comment #3 you meant the checkbox in formatted settings. When I re-rolled the patch I added to the formatter settings checkbox, labeled as 'Uploaded dimensions', if such enabled - then this code fires.

Set to Needs review to get clarification, what further needs to be done.

You're totally right. My apologies. I was rushing through my issue queue reviews and didn't connect the dots even though I was looking right at it. 🤦‍♂️

Leaving in needs review because it needs to be manually tested, with screenshots, etc. Can't guarantee I can get to this level of testing at the moment, so I'm happy to have help from anyone else following this issue.