Merge pull request #24 from Yandex-Practicum/remove-useless-comments

feat: clean useless comments
Yandex-Practicum · Aug 26, 2024 · f0abcce · f0abcce
2 parents 2c549ae + 2a35679
commit f0abcce
Show file tree

Hide file tree

Showing 20 changed files with 0 additions and 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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -183,7 +183,6 @@ function App() {
       <div className="page__content">
         <Header email={email} onSignOut={onSignOut} />
         <Switch>
-          {/*Роут / защищён HOC-компонентом ProtectedRoute*/}
           <ProtectedRoute
             exact
             path="/"
@@ -197,7 +196,6 @@ function App() {
             onCardDelete={handleCardDelete}
             loggedIn={isLoggedIn}
           />
-          {/*Роут /signup и /signin не является защищёнными, т.е оборачивать их в HOC ProtectedRoute не нужно.*/}
           <Route path="/signup">
             <Register onRegister={onRegister} />
           </Route>