[weather] add humidity to hourly view, fix spacing error when using UV Index, add config option to hide precipitation entries that are zero [Rebased Version of PR #3526] #3748
+20
−5
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…(Added Humidity to hourly. Fixed spacing for hourly UV. Added option to hide zeroes in hourly precipitation. Added config option documentation to defaults)
rejas
requested changes
Mar 21, 2025
| @@ -14,7 +14,8 @@ Module.register("weather", { | |||
| updateInterval: 10 * 60 * 1000, // every 10 minutes | |||
| animationSpeed: 1000, | |||
| showFeelsLike: true, | |||
| showHumidity: "none", // this is now a string; see current.njk | |||
| showHumidity: "none", // for current possible options are "none", "wind", "temp", "feelslike" or "below", for hourly "none" or "true" | |||
There was a problem hiding this comment.
Suggested change
| showHumidity: "none", // for current possible options are "none", "wind", "temp", "feelslike" or "below", for hourly "none" or "true" | |
| showHumidity: "none", // possible options for "current" weather are "none", "wind", "temp", "feelslike" or "below", for "hourly" weather "none" or "true" |
|
don't know why but all hourly e2e-weather tests are failing with these changes ... |
|
i havent checked the html elements but test cases get it from |
khassel
requested changes
Mar 21, 2025
| <td class="align-right bright precipitation-amount"> | ||
| {{ hour.precipitationAmount | unit("precip", hour.precipitationUnits) }} | ||
| </td> | ||
| {% if (!config.hideZeroes or hour.precipitationAmount>0) %} |
There was a problem hiding this comment.
{% if (not config.hideZeroes or hour.precipitationAmount>0) %}
| {% endif %} | ||
| {% if config.showPrecipitationProbability %} | ||
| {% if (!config.hideZeroes or hour.precipitationAmount>0) %} |
There was a problem hiding this comment.
{% if (not config.hideZeroes or hour.precipitationAmount>0) %}
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixed Version of PR #3526 since that was against the wrong branch and had issues.
Original PR Text:
Basically the title. Just some existing weather data included into hourly, some config option ("hideZeroes") to hide precipitation when it is zero (this actually shrinks the entire table, removing columns that are completely empty), and add a spacing column to fix the UV Index column.