weather module (standard) - a little typo/error in weather.css?

rkorell

Hi team,

I’m playing around with the standard “weather” module - try to understand, how it works and how I can change something or even develop an own module.
During my journey I had a look into weather.css and there seems to be an error.
It’s possible that my assumption is wrong and that I miss something, than please ignore this…

I’ve found an additional space in line 39

.weather .weathericon,
.weather .fa-home {
  font-size: 75%;
  line-height: 65px;
  display: inline-block;
  transform: translate(0, -3px);
}

.weather .humidity-icon {
  padding-right: 4px;
}

.weather .humidity-padding {
  padding-bottom: 6px;
}

.weather .day {
  padding-left: 0;
  padding-right: 25px;
}

.weather .weather-icon {
  padding-right: 30px;
  text-align: center;
}

.weather .min-temp {
  padding-left: 20px;
  padding-right: 0;
}

.weather .precipitation-amount,
.weather .precipitation-prob,
.weather .uv-index {
  padding-left: 20px;
  padding-right: 0;
}

.weather tr .weathericon {
  line-height: 25px;
}

.weather tr.colored .min-temp {
  color: #bcddff;
}

.weather tr.colored .max-temp {
  color: #ff8e99;
}

this is the block

.weather tr .weathericon {
  line-height: 25px;
}

and the space is between “tr” and the following dot “.” .

this is not the case in the two following blocks.

Regards,
Ralf

sdetweil

@rkorell

With a space means can be on a child element ( child of tr)
Without a space means MUST be on the SAME element(on same tr element)

I do not know the styles or intent of the module, so cannot decide which is correct.

rkorell

@sdetweil OK; good to know.
Thanks!

Warm regards,
Ralf