Please,only take into consideration the last commit:
cd95648

I really don't see the need for this at all.

Bootstrap already applies the following styling to images:

max-width: 100%;
height: auto;

So if no values are defined for the width and height parameters, the image is already repsonsive.

Therefore why do we need to add a paremeter for it?

Agree with @C-Lodder

@C-Lodder , see

• getRandomImage function :
  https://github.com/twister65/joomla-cms/blob/f23cdc3727c847133ae352eb3de9f856e369d79f/modules/mod_random_image/helper.php#L21

• Responsive Random Image Module discussion (on forum) :
  https://forum.joomla.org/viewtopic.php?t=840822

best is:
max-width: 'real_image_width' px;
max-height: 'real_image_height' px;

otherwise small images will get stretched and look ugly

Responsive disabled result (with blank parameters, banner position) :

Responsive enabled result (banner position) :

IMO there really doesn't need to be a parameter. Just remove all the stupid code in the helper.php and use the necessary CSS.

Images should by default, be responsive unless stated otherwise.

helper.php

public static function getRandomImage(&$params, $images)
{
	$width   = $params->get('width') !== null ? $params->get('width') : '';
	$height  = $params->get('height') !== null ? $params->get('height') : '';

	$i      = count($images);
	$random = mt_rand(0, $i - 1);
	$image  = $images[$random];

	$image->width  = $width;
	$image->height = $height;
	$image->folder = str_replace('\\', '/', $image->folder);

	return $image;
}

CSS:

img {
   display: block;
   width: 100%;
   height: auto;
}

done

I have tested this item ? unsuccessfully on 059b53d

tested with protostar in position 8 and position 7

Responsive setting

Not reponnsive with no width set

Not responsive with width set

Cause you need to set the width, not just max-width

@brianteeman , to test you need these commits :

1. cd95648

2. 059b53d

3. f23cdc3

@twister65 We don't test single commits, we always test the complete PR.

Okay, could you explain your testing instructions because I have not the same results.
Test with Protostar in position 7 and position 8.
Image Folder : images/banners

Responsive setting

Not responsive with no width test

Not responsive with width test

I applied the PR
Cleared the caches
Then setup the module as described above

Screenshots on my mobile device (phone).
Position : banner.
With responsive :

Without responsive (and no size) :

I have tested this item ✅ successfully on 059b53d

Patch applied with the latest version of joomla-cms-staging and Pull ID #17688.

@twister65 testing own PR doesn't count cause its expected that a PR is tested by Dev.

I have tested this item ✅ successfully on 059b53d

Without responsive:

With responsive:

I agree with others that this should be default behavior rather than a parameter.

It may be useful to choose, when you want to display random images as thumbnails (with link).

True, perhaps the toggle should be Full Width vs. Thumbnail instead of Responsive vs. Static?

The size is adjusted according to your device (responsive), or not (static).

Please don't add more options. The easiest way to achieve this or anything similar is:

• Create a layout override
• add whatever classes or logic in there
• select between the default or the override you just created

No need for view overrides at all.

Can be done completely just using the current parameters and a CSS/PHP tweak

Disregarding any successful tests, this is just wrong. The code, the method, even the referenced discussion is almost three and a half years old.

@matrikular - so explain what you'd do then

I agree with @matrikular here. The discussion should be on images srcset if we are talking about responsive, adaptive whatever you wanna call it

@dgt41 - That would require the media manager to generate separate image files when uploading. But doesn't solve the actual issue here

If your largest image of the srcset is 1200px, you're using a fluid container, and your viewport is 1920px, it won't be 100$ width unless specified in the CSS

@C-Lodder I'm not sure what you're asking for exactly. You yourself gave some valid answers / examples to that question.

<bla>
There are many different techniques, approaches and caveats to the topic of responsible and / or adaptive images that need to be thoroughly thought through and addressed for the whole site and not seldom, treated differently for different parts and / or content types within a site.

I believe that we can all agree on the fact that changing some inline style attributes for "one" image in "one" module does not accomplish much. The rendered markup, whereever it comes from (CMS / Framework), should be as generic as possible, allowing the site owner / template developer to implement any solution he / she sees fit.

Since the early days, we make use of class suffixes to customise the output and while we added more and more parameters over the years to allow non-developers to change the look and / or behavior of say modules, there is just so much of what can be accomplished (as in limitation) with this approach.

Personally, I'm questioning the module and its implementation. Be it the entry point, the helper or the output.
</bla>

TL/DR
As to the responsiveness; just make sure that width and height are added as an attribute instead of a style declaration. That way, if no container acts as a placeholder, the browser knows how much space the image will use, preventing any ominous flickering / jumping (not that I would have seen such behavior lately). In doing so, you allow "external" style declarations to "take over".

@matrikular All Im asking for is for this parameter to be removed and images be responsive by default unless specified by the width and height parameter.

By your initial comment, I assumed you may have had a different approach.

Set to "closed" on behalf of @Quy by The JTracker Application at issues.joomla.org/joomla-cms/17688

Closing for reasons stated above.