Updating did nothing but gave fatal error warning.

sdetweil

@Richard238 you need to change to the new combined weather module…(announced 2 releases ago)

supposedly change the module name field to weather and change the appid field name to apiKey
custom.css should still be there… I save and restore

Richard238

custom.css is still there, but the coloured icons I had beside calendar entries are no longer showing.

Richard238

@sdetweil Weather module changes worked.
They needs adjusting, but it is now showing something so it’s a good start I can work from.

sdetweil

@Richard238 colored icons

maybe see this
https://forum.magicmirror.builders/topic/16591/mmm-homeassistant-sensors-no-icons-after-mm-upgrade-to-2-19-0

electron is enforcing reported server side (CORS) restrictions is didn’t before , people protecting their content

Richard238

Colour, with a u! I’m British using British English! ;-)

OK, CSS. It’s only using favicon codes, not images on a URL

 #module_3_calendar .fa.fa-calendar-check-o {
  color: #00d255 /* 55=Green FF=Blue*/
}

 #module_4_calendar  .fa.fa-trash-o  {
  color: #C97F7F; /* Brown */
}

sdetweil

@Richard238 said in Updating did nothing but gave fatal error warning.:

#module_3_calendar

u are using ID(#) to select

probably changed locations in the config.js

see https://forum.magicmirror.builders/topic/14862/help-with-a-couple-css-issues?_=1649371486123

Richard238

.calendar .fa.fa-calendar-check-o {
  color: #00d255 /* 55=Green FF=Blue*/
}

.calendar  .fa.fa-trash-o  {
  color: #C97F7F; /* Brown */
}

Makes no difference, the icons still don’t show.

karsten13

@Richard238

what did you define in the symbol: section in the config.js?

May these icons are outdated, you can search here for icons, only free icons will work.

Richard238

I have these, both of which were working perfectly well prior to this update

symbol: 'calendar-check-o ',
symbol: 'trash-o ',

And having configured the new weather module, those icons are responding to custom css just fine.

No idea why calendar icons won’t work.

sdetweil

@Richard238 the trailing space?