Skip to content

Conversation

@skorobaeus
Copy link
Member

Описание

Неожиданно для себя обнаружила, что у любимой демки нет адаптива.

@skorobaeus skorobaeus self-assigned this Oct 27, 2025
@skorobaeus skorobaeus added улучшение Доработка существующего js Контент по JavaScript демо labels Oct 27, 2025
@skorobaeus skorobaeus requested a review from vitya-ne October 27, 2025 19:45
@skorobaeus
Copy link
Member Author

Не могу же я ревьюить сама себя :D

@github-actions
Copy link

Превью контента из опубликовано.

@vitya-ne
Copy link
Contributor

vitya-ne commented Oct 28, 2025

Света, привет!
Демка классная. С адаптивностью теперь всё ок.

У меня несколько предложений по UI:

добавить font-variant-numeric: tabular-nums;

Тогда строчки с временем будет смотреться ровнее:

    #domloaded-time,
    #load-time,
    #difference {
      color: #FFFFFF;
      font-size: 3rem;
      line-height: 1.1;
      font-variant-numeric: tabular-nums;
    }
image

добавить разделитель для большой разницы.

    function countDiff(currentTime) {
      if (!diff) {
        diff = currentTime
      } else {
        diff -= currentTime
        return Math.abs(diff).toLocaleString('ru-RU')
      }
    }

Для больших чисел по-моему красивее смотрится с разделителем тысяч:

image

при тестировании в 3g видно что надписи появляются слишком рано:

Я предлагаю сделать их постепенное появление:

сейчас предлагаю
image image

Copy link
Member

@Inventoris Inventoris left a comment

Choose a reason for hiding this comment

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

Классно, что добралась до адаптива ;-)

По демке в целом меня как будто смущает, что при перезагрузке layout shift происходит у надписей и кнопки, но с другой стороны думаю это не критично в контексте два-три-клик примера.

Copy link
Contributor

@StarHamster StarHamster left a comment

Choose a reason for hiding this comment

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

Меня не звали, а я сам пришёл 🫡

padding: 0;
box-sizing: border-box;
}

Copy link
Contributor

Choose a reason for hiding this comment

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

В демке скроллы есть, их бы покрасить

Suggested change
html {
color-scheme: dark;
}

}

.tip {
margin-top: 40px;
Copy link
Contributor

@StarHamster StarHamster Oct 28, 2025

Choose a reason for hiding this comment

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

Скролл появляется даже на десктопе. При чём при первой загрузке всё ок, а при перезагрузке он появлется. Тут либо айфрем увеличить на 10 пикселей, либо чуть-чуть подрезать воздух кнопке

Suggested change
margin-block-start: 35px;

Часть демки с подписями и кнопкой имеет высоту 430.56. Если кто-то знает, почему при первой загрузке браузер не обращает внимание на эти полпикселя, а потом начинает, объясните мне, пожалуйста 🥹

Comment on lines +149 to +153
.scrollable {
margin-inline-start: 0;
margin-block-start: 40px;
height: 100px;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

На экранах меньше 420 пояляется скролл. А так не появится даже на самых маленьких 💅

Suggested change
.scrollable {
margin-inline-start: 0;
margin-block-start: 40px;
height: 100px;
}
.tip {
margin-top: 30px;
}
.scrollable {
margin-inline-start: 0;
margin-block-start: 30px;
height: 100px;
}

padding: 0;
box-sizing: border-box;
}

Copy link
Contributor

Choose a reason for hiding this comment

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

В демке скроллы есть, их бы покрасить

Suggested change
html {
color-scheme: dark;
}


.tip {
margin-top: 40px;
margin-block-start: 40px;
Copy link
Contributor

Choose a reason for hiding this comment

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

Скролл появляется даже на десктопе. При чём при первой загрузке всё ок, а при перезагрузке он появлется. Тут либо айфрем увеличить на 10 пикселей, либо чуть-чуть подрезать воздух кнопке

Suggested change
margin-block-start: 40px;
margin-block-start: 35px;

Часть демки с подписями и кнопкой имеет высоту 430.56. Если кто-то знает, почему при первой загрузке браузер не обращает внимание на эти полпикселя, а потом начинает, объясните мне, пожалуйста 🥹

Comment on lines +149 to +153
.scrollable {
margin-inline-start: 0;
margin-block-start: 40px;
height: 100px;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

На экранах меньше 420 пояляется скролл. А так не появится даже на самых маленьких 💅

Suggested change
.scrollable {
margin-inline-start: 0;
margin-block-start: 40px;
height: 100px;
}
.tip {
margin-top: 30px;
}
.scrollable {
margin-inline-start: 0;
margin-block-start: 30px;
height: 100px;
}

}

.timings-line + .timings-line {
margin-top: 25px;
Copy link
Contributor

Choose a reason for hiding this comment

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

Давай в обоих демках поменяем на логические?

Suggested change
margin-block-start: 25px;

@Inventoris Inventoris changed the title Добавляет адаптив в демки DOMContentLoaded Добавляет адаптив в демки DOMContentLoaded Nov 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

js Контент по JavaScript демо улучшение Доработка существующего

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants