From 2a35679cf7c0961ed4b0359e002db4acb15b330f Mon Sep 17 00:00:00 2001 From: kotosha Date: Mon, 26 Aug 2024 11:25:53 +0200 Subject: [PATCH] feat: clean useless comments --- .../blocks/card/__delete-button/card__delete-button.css | 1 - .../src/blocks/card/__description/card__description.css | 1 - .../src/blocks/card/__like-button/card__like-button.css | 1 - frontend/src/blocks/card/__title/card__title.css | 1 - frontend/src/blocks/content/content.css | 1 - frontend/src/blocks/footer/footer.css | 1 - frontend/src/blocks/header/header.css | 6 ------ frontend/src/blocks/page/__content/page__content.css | 1 - frontend/src/blocks/page/__section/page__section.css | 1 - frontend/src/blocks/popup/__caption/popup__caption.css | 3 --- frontend/src/blocks/popup/__close/popup__close.css | 3 --- frontend/src/blocks/popup/_is-opened/popup_is-opened.css | 1 - frontend/src/blocks/popup/popup.css | 2 -- .../blocks/profile/__add-button/profile__add-button.css | 1 - .../blocks/profile/__edit-button/profile__edit-button.css | 3 --- frontend/src/blocks/profile/__image/profile__image.css | 2 -- frontend/src/blocks/profile/__info/profile__info.css | 2 -- frontend/src/blocks/profile/__title/profile__title.css | 7 ------- frontend/src/blocks/profile/profile.css | 1 - frontend/src/components/App.js | 2 -- 20 files changed, 41 deletions(-) diff --git a/frontend/src/blocks/card/__delete-button/card__delete-button.css b/frontend/src/blocks/card/__delete-button/card__delete-button.css index 80cf37f5..5b950e4e 100644 --- a/frontend/src/blocks/card/__delete-button/card__delete-button.css +++ b/frontend/src/blocks/card/__delete-button/card__delete-button.css @@ -2,7 +2,6 @@ width: 18px; height: 18px; background: transparent url('../../../images/delete-icon.svg') center no-repeat; - /*Потом это будет из модификатора*/ background-size: 18px 18px; border: none; cursor: pointer; diff --git a/frontend/src/blocks/card/__description/card__description.css b/frontend/src/blocks/card/__description/card__description.css index cb9e50c2..3c430cd5 100644 --- a/frontend/src/blocks/card/__description/card__description.css +++ b/frontend/src/blocks/card/__description/card__description.css @@ -4,5 +4,4 @@ align-items: center; background: #fff; padding: 25px 20px; - /*Чтобы по ПЭПЭ отступы снизу-сверху одинаковые */ } diff --git a/frontend/src/blocks/card/__like-button/card__like-button.css b/frontend/src/blocks/card/__like-button/card__like-button.css index 57a7db07..2a5c4cc4 100644 --- a/frontend/src/blocks/card/__like-button/card__like-button.css +++ b/frontend/src/blocks/card/__like-button/card__like-button.css @@ -2,7 +2,6 @@ width: 21px; height: 18px; background: transparent url('../../../images/like-inactive.svg') no-repeat; - /*Потом это будет из модификатора*/ background-size: 21px 18px; border: none; cursor: pointer; diff --git a/frontend/src/blocks/card/__title/card__title.css b/frontend/src/blocks/card/__title/card__title.css index c020f1b2..d899fcc9 100644 --- a/frontend/src/blocks/card/__title/card__title.css +++ b/frontend/src/blocks/card/__title/card__title.css @@ -5,6 +5,5 @@ text-overflow: ellipsis; white-space: nowrap; overflow: hidden; - /*Либо с точками из макета, либо этот набор свойств для эллипсиса текста в можно лучше*/ margin: 0; } diff --git a/frontend/src/blocks/content/content.css b/frontend/src/blocks/content/content.css index 180ff404..84ac9e4d 100644 --- a/frontend/src/blocks/content/content.css +++ b/frontend/src/blocks/content/content.css @@ -1,5 +1,4 @@ .content { flex-shrink: 0; flex-grow: 1; - /*Стики футер*/ } diff --git a/frontend/src/blocks/footer/footer.css b/frontend/src/blocks/footer/footer.css index 6aa858e3..9455f1b2 100644 --- a/frontend/src/blocks/footer/footer.css +++ b/frontend/src/blocks/footer/footer.css @@ -3,7 +3,6 @@ .footer { font-family: 'Inter', Arial, sans-serif; flex-shrink: 0; - /*Стики футер*/ padding: 30px 0 60px; } diff --git a/frontend/src/blocks/header/header.css b/frontend/src/blocks/header/header.css index a218accb..80611d86 100644 --- a/frontend/src/blocks/header/header.css +++ b/frontend/src/blocks/header/header.css @@ -6,14 +6,10 @@ .header { min-height: 120px; - /*Если подгонять под ПЭПЭ, то тут min-height: 117px;*/ font-family: 'Inter', Arial, sans-serif; display: flex; align-items: center; - /*border-bottom: 1px solid rgba(84, 84, 84, 0.7);*/ - /*Перевести в RGBA, чтобы учесть opacity*/ flex-shrink: 0; - /*Стики футер*/ justify-content: space-between; } @@ -30,10 +26,8 @@ -ms-transform: translateX(-50%); -o-transform: translateX(-50%); transform: translateX(-50%); - /*Центруем. Студенты знают про транслейт*/ bottom: 0; } -/*Лучшее решение здесь -- можно лучше*/ @media screen and (max-width: 480px) { .header { diff --git a/frontend/src/blocks/page/__content/page__content.css b/frontend/src/blocks/page/__content/page__content.css index d394863c..91bc1443 100644 --- a/frontend/src/blocks/page/__content/page__content.css +++ b/frontend/src/blocks/page/__content/page__content.css @@ -5,5 +5,4 @@ -moz-osx-font-smoothing: grayscale; display: flex; flex-direction: column; - /*Уже пора бы в можно лучше про стики-футер рассказать*/ } diff --git a/frontend/src/blocks/page/__section/page__section.css b/frontend/src/blocks/page/__section/page__section.css index fe6ff0c6..460c63d3 100644 --- a/frontend/src/blocks/page/__section/page__section.css +++ b/frontend/src/blocks/page/__section/page__section.css @@ -1,7 +1,6 @@ .page__section { width: 100%; max-width: 882px; - /*Опционально. Тут можно принять проект и без ограничения максимальной ширины.*/ -webkit-box-sizing: border-box; -moz-box-sizing: border-box; box-sizing: border-box; diff --git a/frontend/src/blocks/popup/__caption/popup__caption.css b/frontend/src/blocks/popup/__caption/popup__caption.css index 5e127049..35ef2b88 100644 --- a/frontend/src/blocks/popup/__caption/popup__caption.css +++ b/frontend/src/blocks/popup/__caption/popup__caption.css @@ -6,7 +6,4 @@ left: 0; top: calc(100% + 10px); margin: 0; - /* - позиционируем абсолютно. В задании сказано только про максимальные размеры изображения. - */ } diff --git a/frontend/src/blocks/popup/__close/popup__close.css b/frontend/src/blocks/popup/__close/popup__close.css index 1a625c3a..db3a2db6 100644 --- a/frontend/src/blocks/popup/__close/popup__close.css +++ b/frontend/src/blocks/popup/__close/popup__close.css @@ -2,9 +2,7 @@ width: 35px; height: 35px; background: transparent url('../../../images/close.svg') center no-repeat; - /*Шортхэнды пора бы знать, хотя бы в можно лучше*/ background-size: 35px 35px; - /*Можно марджином тут выровнять под ПЭПЭ, но тут оставил AS.*/ border: none; position: absolute; top: -36px; @@ -15,7 +13,6 @@ -o-transform: rotate(90deg); transform: rotate(90deg); transition: visibility 0s, opacity 0.3s; - /*Транзишн пора бы уже знать*/ padding: 0; margin: 0; cursor: pointer; diff --git a/frontend/src/blocks/popup/_is-opened/popup_is-opened.css b/frontend/src/blocks/popup/_is-opened/popup_is-opened.css index 9c32dad5..207e395e 100644 --- a/frontend/src/blocks/popup/_is-opened/popup_is-opened.css +++ b/frontend/src/blocks/popup/_is-opened/popup_is-opened.css @@ -2,6 +2,5 @@ visibility: visible; opacity: 1; pointer-events: all; - /* When opening, make the visibility change immediate with no delay */ transition: visibility 0s, opacity 0.6s; } diff --git a/frontend/src/blocks/popup/popup.css b/frontend/src/blocks/popup/popup.css index 6d7da41c..c5f3b2cb 100644 --- a/frontend/src/blocks/popup/popup.css +++ b/frontend/src/blocks/popup/popup.css @@ -28,12 +28,10 @@ left: 0; bottom: 0; right: 0; - /*Стили для плавного появления модалки*/ visibility: hidden; opacity: 0; pointer-events: none; user-select: none; - /* Delay visibility: hidden until the end of the opacity animation, but only on the fade-out direction */ transition: visibility 0s 0.6s, opacity 0.6s; z-index: 10; } diff --git a/frontend/src/blocks/profile/__add-button/profile__add-button.css b/frontend/src/blocks/profile/__add-button/profile__add-button.css index 93979963..06dee3d4 100644 --- a/frontend/src/blocks/profile/__add-button/profile__add-button.css +++ b/frontend/src/blocks/profile/__add-button/profile__add-button.css @@ -6,7 +6,6 @@ -webkit-border-radius: 2px; -moz-border-radius: 2px; border-radius: 2px; - /*Че это*/ border: 2px solid #fff; -webkit-transition: 0.3s; -moz-transition: 0.3s; diff --git a/frontend/src/blocks/profile/__edit-button/profile__edit-button.css b/frontend/src/blocks/profile/__edit-button/profile__edit-button.css index 23209213..9816cdf1 100644 --- a/frontend/src/blocks/profile/__edit-button/profile__edit-button.css +++ b/frontend/src/blocks/profile/__edit-button/profile__edit-button.css @@ -2,19 +2,16 @@ width: 24px; height: 24px; background: transparent url('../../../images/edit-icon.svg') center no-repeat; - /*Шортхэнды пора бы знать, хотя бы в можно лучше*/ background-size: 10px 10px; border: 1px solid #fff; grid-area: button; align-self: center; - /*Можно марджином тут выровнять под ПЭПЭ, но тут оставил AS.*/ cursor: pointer; -webkit-transition: 0.3s; -moz-transition: 0.3s; -ms-transition: 0.3s; -o-transition: 0.3s; transition: 0.3s; - /*Транзишн пора бы уже знать*/ padding: 0; margin: 0; } diff --git a/frontend/src/blocks/profile/__image/profile__image.css b/frontend/src/blocks/profile/__image/profile__image.css index 41710ded..a8c43084 100644 --- a/frontend/src/blocks/profile/__image/profile__image.css +++ b/frontend/src/blocks/profile/__image/profile__image.css @@ -5,11 +5,9 @@ -moz-border-radius: 50%; border-radius: 50%; background-size: cover; - /*Ковер тут симпатичнее контейна*/ background-position: center; position: relative; margin: 0 29px 0 0; - /*Макет очень кривоват, как считать этот отступ я хз. Проявите тут лояльность пока что*/ } .profile__image:hover { diff --git a/frontend/src/blocks/profile/__info/profile__info.css b/frontend/src/blocks/profile/__info/profile__info.css index 3c0bacd5..7aaefee7 100644 --- a/frontend/src/blocks/profile/__info/profile__info.css +++ b/frontend/src/blocks/profile/__info/profile__info.css @@ -3,9 +3,7 @@ grid-template-areas: "title button" "description description"; grid-template-columns: minmax(auto, 295px) auto; - /*Можно не гридить*/ grid-gap: 9px 17px; - /*Подгон под ПЭПЭ, без него - 16px*/ } @media screen and (max-width: 740px) { diff --git a/frontend/src/blocks/profile/__title/profile__title.css b/frontend/src/blocks/profile/__title/profile__title.css index e7709440..4c7e2ddf 100644 --- a/frontend/src/blocks/profile/__title/profile__title.css +++ b/frontend/src/blocks/profile/__title/profile__title.css @@ -1,12 +1,10 @@ .profile__title { font-size: 42px; line-height: 48px; - /*Подгон под ПЭПЭ - без него 56px*/ font-weight: 400; text-overflow: ellipsis; white-space: nowrap; overflow: hidden; - /*Либо с точками из макета, либо этот набор свойств для эллипсиса текста в можно лучше*/ grid-area: title; margin: 0; } @@ -22,16 +20,11 @@ .profile__title { font-size: 27px; line-height: 33px; - /*По макету 56px. Проявите лоялити к студенту*/ } } @media screen and (max-width: 480px) { .profile__title { min-width: 201px; - /* - Это по макету. Вообще я считаю тут должен быть отступ до иконки, но фрейм у заголовка кривой. - Проявите лоялити к студенту - */ } } diff --git a/frontend/src/blocks/profile/profile.css b/frontend/src/blocks/profile/profile.css index 6e856866..01d738e7 100644 --- a/frontend/src/blocks/profile/profile.css +++ b/frontend/src/blocks/profile/profile.css @@ -11,7 +11,6 @@ display: flex; align-items: center; padding: 36px 0; - /*Если подгонять под ПЭПЭ, то тут padding: 40px 0; Проявите лоялити*/ } @media screen and (max-width: 480px) { diff --git a/frontend/src/components/App.js b/frontend/src/components/App.js index 88c8ac54..f950da2f 100644 --- a/frontend/src/components/App.js +++ b/frontend/src/components/App.js @@ -183,7 +183,6 @@ function App() {
- {/*Роут / защищён HOC-компонентом ProtectedRoute*/} - {/*Роут /signup и /signin не является защищёнными, т.е оборачивать их в HOC ProtectedRoute не нужно.*/}