Skip to content

Conversation

@Zorin-Alex
Copy link

No description provided.

Copy link

@aleksandrbelik aleksandrbelik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to add support IE 11. It can be implemented using special prefixes -ms. Please read about IE grid supporting.
Also with flex text "28 C" on small mobiles goes on second row

<body>
<article class="weather-widget">
<div class="weather-widget__location">
<div class="weather-widget__weather-type-logo"><img src="img/weather/sunny-big.png" width="50%"></div>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Zorin-Alex It's better to set width style in css too. You can just add class to <img> tag and use it in css file

<div class="weather-widget__weekday-information"><p class="weather-widget__weekday">Fri</p><p class="weather-widget__daytime-temp">22 &deg;C</p><p class="weather-widget__nighttime-temp">20 &deg;C</p></div>
</div>
<div class="weather-widget__day">
<img class="weather-widget__day-weather-logo" src="img/weather/cloudy.png" width="70%">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Zorin-Alex Same case with img and it's width for these three .weather-widget__day-weather-logo elements

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants