Merge pull request #1838 from tbouron/fix/module-header

Module header updates correctly, if a module need to dynamically show/hide its header based on a condition
This commit is contained in:
Michael Teeuw 2019-12-28 19:41:42 +01:00 committed by GitHub
commit 05f710cb5c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 9 deletions

View File

@ -27,6 +27,7 @@ This project adheres to [Semantic Versioning](http://semver.org/).
### Fixed ### Fixed
- Fixed issue in weatherforecast module where predicted amount of rain was not using the decimal symbol specified in config.js. - Fixed issue in weatherforecast module where predicted amount of rain was not using the decimal symbol specified in config.js.
- Module header now updates correctly, if a module need to dynamically show/hide its header based on a condition.
## [2.9.0] - 2019-10-01 ## [2.9.0] - 2019-10-01

View File

@ -39,11 +39,13 @@ var MM = (function() {
dom.opacity = 0; dom.opacity = 0;
wrapper.appendChild(dom); wrapper.appendChild(dom);
if (typeof module.getHeader() !== "undefined" && module.getHeader() !== "") { var moduleHeader = document.createElement("header");
var moduleHeader = document.createElement("header"); moduleHeader.innerHTML = module.getHeader();
moduleHeader.innerHTML = module.getHeader(); moduleHeader.className = "module-header";
moduleHeader.className = "module-header"; dom.appendChild(moduleHeader);
dom.appendChild(moduleHeader);
if (typeof module.getHeader() === "undefined" || module.getHeader() !== "") {
moduleHeader.style = "display: none;";
} }
var moduleContent = document.createElement("div"); var moduleContent = document.createElement("div");
@ -210,9 +212,8 @@ var MM = (function() {
contentWrapper[0].innerHTML = ""; contentWrapper[0].innerHTML = "";
contentWrapper[0].appendChild(newContent); contentWrapper[0].appendChild(newContent);
if( headerWrapper.length > 0 && newHeader) { headerWrapper[0].innerHTML = newHeader;
headerWrapper[0].innerHTML = newHeader; headerWrapper[0].style = headerWrapper.length > 0 && newHeader ? undefined : "display: none;";
}
}; };
/* hideModule(module, speed, callback) /* hideModule(module, speed, callback)