4.7.6. – added: alt attribute to fixed size masonry background image container

no – I thought I had found a solution for this, but it was not the right approach

Edit : the approach was right – but trying to transfer the code to the minified version was bad:
i added an else if to the code:

var el = template.find('.mfp-img');
if(el.length) {
	var img = document.createElement('img');
	img.className = 'mfp-img';
	if(item.el && item.el.find('img').length) {
		img.alt = item.el.find('img').attr('alt');
	}
	else if(item.el && item.el.find('.av-masonry-image-container').length) {
		img.alt = item.el.find('.av-masonry-image-container').attr('alt');
	}
	item.img = $(img).on('load.mfploader', onLoadComplete).on('error.mfploader', onLoadError);
	img.src = item.src;

that works ( i used a different minifier on that ): https://webers-testseite.de/pureinstall/masonry-gallery/

By the way: what kind of minifier do you use for enfold

Vielen Lieben Dank für dein Bemühen, aber leider klappt es so nicht.

Wobei ich nicht sehe, warum es so nicht geht.

In meinem custom magnific popup script benutze ich nun auch den fakt , dass am item selbst immer das alt attribut sitzt. Das hattest du ja schon geändert. Deshalb kann man auch hier link bei allen Masonries das Alt-Attribute im Markup als Text sehen. Aber am lightbox image selber ist das Alt-Attribut nicht zu sehen.
Damit ich ausschließen konnte, dass es an meinem Custom Script (Seitenspezifisch) liegt habe ich hier eine Seite wo das standard Enfold Verhalten ist: Link
mit den Veränderungen oben von mir im jquery.magnific-popup.js war es sofort da.

ja Du hast recht – sorry dafür. Die habe ich übersehen, und im Standard Masonry auch fixed geht es jetzt.

In meinem custom magnific script hatte ich titleSrc auf false gesetzt – was deiner Lösung den Gar ausmachte.
Es ist aber so, dass die Kombination deiner beiden Austausch Dateien mit dem custom script so nicht dazu führt das alt am mfp-img zu haben.
Ich für mich werde dann wohl beide lösungen anstreben müssen.

Kann geschlossen werden –
PS wird das im nächsten Update integriert?

• This reply was modified 4 years, 3 months ago by Guenni007.