MMM-BMW-DS (A different kind of weather module)

Mykle1

@sdetweil said in MMM-BMW-DS (A different kind of weather module):

i just installed this module, and the default CSS causes the display to take over the entire screen…
css:1, fixes it… I ‘assume’ the default should be 1?

Sorry the module is giving you trouble. You should not have a “default.css” file at all. In the css folder of the module there should only be 6 css files, named:

MMM-BMW-DS1.css
MMM-BMW-DS2.css
MMM-BMW-DS3.css
MMM-BMW-DS4.css
MMM-BMW-DS5.css
MMM-BMW-DS6.css

Maybe the way I implemented this, or documented this, is not very clear. I do apologize for that. If you have a suggestion for cleaning this up I would appreciate that, or if the problem is something else then please assist me in understanding and correcting.

sdetweil

your readme

  css: "1",                         // 1-6 (default, Clean, Lord of the Rings, Handwriting, etc)

ok, what is ‘default’?

ie, the setting if one doesn’t chose anything?

if I do not supply the line above, what value will be used

the screen shots show a line at the bottom of the page, so I ASSUME this must be the ‘default’ behavior…

Mykle1

@sdetweil said in MMM-BMW-DS (A different kind of weather module):

if I do not supply the line above, what value will be used

Ok, I see. I’m off to dinner right now but I will correct that when I return. I’m sorry for the trouble and thank you for pointing that out to me.

Mykle1

@sdetweil

Ok, this is what I’ve done. I don’t know if this is the proper way of doing it but it does work.

// Gets correct css file from config.js
    getStyles: function() {
      if(this.config.css != ""){
        return ["modules/MMM-BMW-DS/css/MMM-BMW-DS" + this.config.css + ".css"];
      } else {
        return ["modules/MMM-BMW-DS/css/MMM-BMW-DS1.css"]; // default.css
      }
    },

sdetweil

@mykle1 easier if u just declare a defaults section in your module.js which has css: 1

The when it is merged with the user supplied config, there will always be something set.

Then u don’t need the if/then at all

Mykle1

@sdetweil

I don’t know how I missed that. Thanks again for showing me the error of my ways. :-)

// Module config defaults.
    defaults: {
        apiKey: "", // Get FREE API key from darksky.net
        tempUnits: "C", // C  or F
        lat: "40.123456", // Latitude
        lng: "-74.123456", // Longitude
        css: "1", // 1=default, 2=Clean, 3=Lord of the Rings, 4=handwriting, 5=Julee, 6=Englebert
        ownTitle: "Current Conditions", // Default = Current Conditions
        useHeader: false, // true if you want a header
        header: "Your Header", // Any text you want. useHeader must be true
        maxWidth: "100%",
        animationSpeed: 3000,
        initialLoadDelay: 4250,
        retryDelay: 2500,
        updateInterval: 5 * 60 * 1000,

cirdan

Mykle, kids called me at work today. magic mirror is making weird noises. 😀

Afraid for hardware failure I went home only to find out mmmbmwds had added sounds and it rained for the first time.

Would you consider building in a config switch to enable/disable sounds?

sdetweil

Rename the mp3 files to something else

Mykle1

@cirdan said in MMM-BMW-DS (A different kind of weather module):

Would you consider building in a config switch to enable/disable sounds?

git pull in the MMM-BMW-DS directory and playSounds: "no" in the new config option. :thumbsup:

Why can’t everything I try to do with modules be that easy?

cirdan

@mykle1 tnx for the quick fix.
(At last peace returns to the grey havens 😁)