Calendar moves right by 5 px every update

[sdetweil]

@emwtur the fix was when there were multiple symbols

symbol:[x,y,z]

in the calendar specification

[emwtur]

@sdetweil
Yes I had multiple but tried to remove all of them - didn’t help.

Anyway having ONLY the cal from config.js.sample looks to work - having restored calendar.js to original

[emwtur]

@sdetweil
it has to do with multiple symbols.
looks like the bug can be reproduced using the cal setting from the sample file , just add another symbol and watch it move

{
		module: "calendar",
		header: "US Holidays",
		position: "top_left",
		config: {
			calendars: [
				{
					fetchInterval: 7 * 24 * 60 * 60 * 1000,
					symbol: ["calendar-check","google"]
					url: "https://ics.calendarlabs.com/76/mm3137/US_Holidays.ics"
				}
			]
		}
	},

[sdetweil]

@emwtur great!!! thsnks

[emwtur]

@sdetweil

Having single symbols is something I can live with - was just confused - it worked before updating…

thank YOU for great work with Magic Mirror.
regards
/Per-Eric

[sdetweil]

@emwtur ok, i made the change , so gotta figure out how i screwed that up

[sdetweil]

@emwtur want to try the fix

calendar.js , line 915

			if (p instanceof Array) {   // keep add brace, remove rest of line or // comment out
			 let temp=[]
			 p.forEach(name=>{temp.push(className +name)});
			 p = temp
			}                                                  
			else p = className + p;  // keep

[emwtur]

@sdetweil
Looks like it’s working with that fix - line numbers didn’t match so I had to think a little before applying - tested with multiple symbols on some cals and both symbols show and stay in place

/Per-Eric

[sdetweil]

@emwtur awesome!!!

[sdetweil]

@emwtur the fix for this has been accepted and added to the next release (develop branch)

develop includes the fix for broadcast clipping too…
if you have the develop branch, then you can git pull to update

if you don’t have develop, then see this topic
https://forum.magicmirror.builders/topic/14327/testing-new-fixes-or-solving-current-problems-with-next-release-code