MMM-Flights

[karsten13]

@lif

was talking about showMap: "ifTraffic", and this works on my side, tested it again

[lif]

@karsten13 said in MMM-Flights:

@lif

was talking about showMap: "ifTraffic", and this works on my side, tested it again

Yes

 showMap: "ifTraffic", // "always", "never", "ifTraffic"

[sdetweil]

@lif said in MMM-Flights:

I assume that it can be changed in the {module}.js file.

no.

in config.js

the design is config js is used to override the values set in the modulename.js defaults section

lacking documentation is a different probem.

if you find properties in the defaults section you can override them in config.js

[sdetweil]

@lif I have a module that presents a form for config customization, built on the fly based on the modules you have installed now

it uses the values from defaults

https://github.com/sdetweil/MMM-Config

in places where there could be choices, the author can augment the definition to make it more usable and understandable.

not every module works some need help.
if the module is old or the author chooses not to provide an improved definition, I can ship one created by a user, with my module to make the user experience better.

[lif]

@sdetweil said in MMM-Flights:

@lif I have a module that presents a form for config customization, built on the fly based on the modules you have installed now

it uses the values from defaults

https://github.com/sdetweil/MMM-Config

in places where there could be choices, the author can augment the definition to make it more usable and understandable.

not every module works some need help.
if the module is old or the author chooses not to provide an improved definition, I can ship one created by a user, with my module to make the user experience better.

Nice advert :-) but I’ve never been able to cut and paste, in some cases , 20 - 30 lines of options from the module config file to config.js using a command line editor if the author doesn’t give all the options in the default config.js example on Git Hub.

[sdetweil]

@lif it’s a fun challenge.

the downside of changing the modulename.js file is, when an update comes, git pull will fail. and YOU have to figure it out.

everything takes work. I’m just trying to help you learn the best practice so you don’t get smashed later on.

[stashu]

Hi! Thank you for the module, it is my favorite on my MM, as I live on the flight path to KIAD, so lots of activity.

I am wondering if there is a way I can hide the horizontal line which is under the ‘Flights’ label. When there are no flights, the line is still visible, and I would love if I were able to hide it somehow. I’m happy to add something to custom.css, but I am unsure how to do so. Thanks!

[sdetweil]

@stashu So, there is indeed custom.css entry you can use the hide the separator on the header…

BUT I want to teach you how to find these things yourself… so you won’t be dependant on waiting for someone

see the second link in my signature below, it will show you how to use the developers window elements tab and locate the element that is displaying the thing you want to modify/hide…

in the upper right window you can try different styles on the selected element to see its behavior and once you’ve found it, you can copy paste those styles into custom.css
and put the selector wrapper around it.

the selector chain is shown on the bottom of the elements window , for example here on another element

[stashu]

@sdetweil thanks for the quick reply and the pointer to finding the element. I’m not in web stuff, so it’s a bit more of a mystery to me than I wish it were. Anyway, I found it, and the following custom.css entry works to remove the header:

.MMM-Flights .module-header {
	border-bottom: 0px; /* removes line */
	font-size: 0px; /* removes 'Flights' and logo (?) */
}

Not sure if that’s an elegant way, but it works :) . I’m not clear on why the 0 font size removes the logo, but it does.

Thanks again!

[sdetweil]

@stashu all good… sorry for the firehose treatment… lol

anyhow to hide an element is actually easier

selector {
   display:none;
}