In \plugins\fields\imagelist\tmpl\imagelist.php, directory is accounted for. Does it apply here? If yes, probably should be in a separate PR.

	if (!$path || $path == '-1')
	{
		continue;
	}

	if ($fieldParams->get('directory', '/') !== '/')
	{
		$buffer .= sprintf('<img src="images/%s/%s"%s>',
			$fieldParams->get('directory'),
			htmlentities($path, ENT_COMPAT, 'UTF-8', true),
			$class
		);
	}
	else
	{
		$buffer .= sprintf('<img src="images/%s"%s>',
			htmlentities($path, ENT_COMPAT, 'UTF-8', true),
			$class
		);
	}

I have tested this item ✅ successfully on a1e0b51

There is a lot more than just removing the slash in this PR

I don't see where the / is added

@PhilETaylor ok, then I'm really confuzzled because then we're breaking html?

html doesn’t require the use of a closing slash to close void tags - a quick google will have you thinking you were mad to think it did, and back again. Its optional apparently, but there are those in the joomla world now demanding that PR creators are held to different standards than those contributing a whole new feature (com_fields) apparently.

On 22 May 2017, at 21:06, Bear notifications@github.com wrote:

@PhilETaylor https://github.com/philetaylor ok, then I'm really confuzzled because then we're breaking html?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub #16151 (comment), or mute the thread https://github.com/notifications/unsubscribe-auth/AAYa3LNt9sSrJs50iqqWKraLYf0GXnZ8ks5r8erPgaJpZM4NhXsD.

@N6REJ no we are not. html5 does not require the /

@PhilETaylor I think I'll go back to contemplating why the hamster spins the wheel ... thanks

With and -out PR got:

System information

3.7.3-dev
Multilanguage Site
macOS Sierra, 10.12.4
Firefox 53 (64-bit)

MAMP 4.1.1

• PHP 7.0.15
• MySQLi 5.6.35

@franz-wohlkoenig
This works for an imagelist custom field.
I get here OK

dl class="fields-container">
			<dd class="field-entry ">
			<span class="field-label">imagelist: </span>
<span class="field-value"><img src="/testnew/trunkgitnew/images/joomla_black.png" class="imgclass"></span>
	</dd>
</dl>

and for media

<dd class="field-entry ">
			<span class="field-label">media: </span>
<span class="field-value"><img src="/testnew/trunkgitnew/images/powered_by.png" class="mediaclass"></span>
	</dd>

I have tested this item ✅ successfully on a1e0b51

@franz-wohlkoenig that is impossible as there is no closing hash in the source code after the PR is applied. What I have seen though, is that Google Chrome Dev Console (and others) likes to fake the html to make it "more valid" and so sometimes adds tags. Try viewing the source code of the page using "View Source Code" features of your browser instead of using the dev tools panel.

RTC after two successful tests.

@infograf768 didn't saw its about com_fields.