HEIC and MMM-ImagesPhotos

[sdetweil]

@rts58 can you look in the developers window console tab

ctrl-shift-i

[rts58]

@sdetweil yes I’ve been looking at it. If it works I’ll get a message that the image loaded and link, image setting size … , and setting top. If it doesn’t it won’t show anything.

[sdetweil]

@rts58 ok, i’ll look at error handlers

[rts58]

@sdetweil if it helps when I dig into the elements it shows:

<img src="/MMM-ImagesPhotos/photo/module_9_MMM-ImagesPhotos/IMG_3259-Copy.HEIC" style="left: 0px; position: relative; opacity: 0;">

So it thinks it is loading the file.

[sdetweil]

@rts58

in front of

img.onload()

add

img.onerror(){
   this.updateDom()
}

[rts58]

@sdetweil this is what I did which is creating an uncaught syntax error on the line with the first “{”.

	img.onerror(){
	this.updateDom()
	}
        img.onload = (evt) => {
          // Get the image of the event
          const eventImage = evt.currentTarget;
          Log.log(
            `image loaded=${eventImage.src} size=${eventImage.width}:${eventImage.height}`

[sdetweil]

@rts58 im an idiot, sorry

better

img.onerror= ()=>{
	this.updateDom()
}

[rts58]

@sdetweil I’m not seeing any errors in the console.

[sdetweil]

@rts58 i did not add error reporting yet, this should keep it
from stalling

[sdetweil]

@rts58 the prior code started the timer only after
image was loaded. but we have already waited timeout time
so we can get the next image

[sdetweil]

@rts58 just pushed the updated code, git pull in the module folder
logs an error record to the dev window console on load fail, with the url of the failing image

[rts58]

@sdetweil great, thanks. I’ll let you know how it goes.

[rts58]

@sdetweil It’s working great now. No longer freezing on image files that it cannot open. And it is properly reflecting the error with the file name in the console! THANKS!

[sdetweil]

@rts58 awesome!!