first commit #1
first commit #1
Conversation
There was a problem hiding this comment.
при переходе на страницу таймер отображается не сразу и высота контейнера зависит от его наличия.
поправь верстку, чтобы соответствовала макету (шрифты, отступы и тд);
нужно, чтобы роутинг был без хэшей #;
при обновлении неверно отображается активная страница;
при переходе между страницами с помощью кнопок назад/вперед в браузере, не изменяется подсветка активной страницы;
Исправишь - тегни меня здесь
There was a problem hiding this comment.
дублируешь иконки
можно экспортировать все иконки из фигмы
используй svg формат, он легче
| @@ -0,0 +1,228 @@ | |||
| const mapPage = ` | |||
There was a problem hiding this comment.
интересный подход, но мб вынести html к html? в js файлах оставить только js
|
|
||
| hours = (hours < 10) ? "0" + hours : hours; | ||
| minutes = (minutes < 10) ? "0" + minutes : minutes; | ||
| seconds = (seconds < 10) ? "0" + seconds : seconds; |
There was a problem hiding this comment.
можно как-нибудь избежать дублирования?)
| } | ||
|
|
||
| function initializeMap() { | ||
| setTimeout(() => { |
| navigateTo('/timer'); | ||
| }); | ||
|
|
||
| document.querySelectorAll('nav ul li').forEach(function(item) { |
There was a problem hiding this comment.
выше везде писал через () => {
выбери один подход
| @@ -0,0 +1,15 @@ | |||
| .container, .row{ | |||
| max-width: 760px; | |||
There was a problem hiding this comment.
задавай размеры за счет margin/padding + у тебя же адаптив должен быть
There was a problem hiding this comment.
еще я не могу понять откуда ты берешь иконки?))
экспортируй из фигмы
There was a problem hiding this comment.
можно просто использовать теги <link> и <script> (см. доку bootstrap)
| <i class="fas fa-camera me-1"></i> <span>Photos</span> | ||
| </div> | ||
| <div class="icon-text-group"> | ||
| <i class="fas fa-cog me-1"></i> <span></span> |
No description provided.