Skip to content

Commit

Permalink
feat: clean useless comments
Browse files Browse the repository at this point in the history
  • Loading branch information
kotosha-real committed Aug 22, 2024
1 parent 2c549ae commit 474ff67
Show file tree
Hide file tree
Showing 21 changed files with 6,259 additions and 16,749 deletions.
22,967 changes: 6,259 additions & 16,708 deletions frontend/package-lock.json

Large diffs are not rendered by default.

Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,5 +4,4 @@
align-items: center;
background: #fff;
padding: 25px 20px;
/*Чтобы по ПЭПЭ отступы снизу-сверху одинаковые */
}
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
1 change: 0 additions & 1 deletion frontend/src/blocks/card/__title/card__title.css
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,5 @@
text-overflow: ellipsis;
white-space: nowrap;
overflow: hidden;
/*Либо с точками из макета, либо этот набор свойств для эллипсиса текста в можно лучше*/
margin: 0;
}
1 change: 0 additions & 1 deletion frontend/src/blocks/content/content.css
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
.content {
flex-shrink: 0;
flex-grow: 1;
/*Стики футер*/
}
1 change: 0 additions & 1 deletion frontend/src/blocks/footer/footer.css
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
.footer {
font-family: 'Inter', Arial, sans-serif;
flex-shrink: 0;
/*Стики футер*/
padding: 30px 0 60px;
}

Expand Down
6 changes: 0 additions & 6 deletions frontend/src/blocks/header/header.css
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}

Expand All @@ -30,10 +26,8 @@
-ms-transform: translateX(-50%);
-o-transform: translateX(-50%);
transform: translateX(-50%);
/*Центруем. Студенты знают про транслейт*/
bottom: 0;
}
/*Лучшее решение здесь -- можно лучше*/

@media screen and (max-width: 480px) {
.header {
Expand Down
1 change: 0 additions & 1 deletion frontend/src/blocks/page/__content/page__content.css
Original file line number Diff line number Diff line change
Expand Up @@ -5,5 +5,4 @@
-moz-osx-font-smoothing: grayscale;
display: flex;
flex-direction: column;
/*Уже пора бы в можно лучше про стики-футер рассказать*/
}
1 change: 0 additions & 1 deletion frontend/src/blocks/page/__section/page__section.css
Original file line number Diff line number Diff line change
@@ -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;
Expand Down
3 changes: 0 additions & 3 deletions frontend/src/blocks/popup/__caption/popup__caption.css
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,4 @@
left: 0;
top: calc(100% + 10px);
margin: 0;
/*
позиционируем абсолютно. В задании сказано только про максимальные размеры изображения.
*/
}
3 changes: 0 additions & 3 deletions frontend/src/blocks/popup/__close/popup__close.css
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand All @@ -15,7 +13,6 @@
-o-transform: rotate(90deg);
transform: rotate(90deg);
transition: visibility 0s, opacity 0.3s;
/*Транзишн пора бы уже знать*/
padding: 0;
margin: 0;
cursor: pointer;
Expand Down
1 change: 0 additions & 1 deletion frontend/src/blocks/popup/_is-opened/popup_is-opened.css
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
2 changes: 0 additions & 2 deletions frontend/src/blocks/popup/popup.css
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down
2 changes: 0 additions & 2 deletions frontend/src/blocks/profile/__image/profile__image.css
Original file line number Diff line number Diff line change
Expand Up @@ -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 {
Expand Down
2 changes: 0 additions & 2 deletions frontend/src/blocks/profile/__info/profile__info.css
Original file line number Diff line number Diff line change
Expand Up @@ -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) {
Expand Down
7 changes: 0 additions & 7 deletions frontend/src/blocks/profile/__title/profile__title.css
Original file line number Diff line number Diff line change
@@ -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;
}
Expand All @@ -22,16 +20,11 @@
.profile__title {
font-size: 27px;
line-height: 33px;
/*По макету 56px. Проявите лоялити к студенту*/
}
}

@media screen and (max-width: 480px) {
.profile__title {
min-width: 201px;
/*
Это по макету. Вообще я считаю тут должен быть отступ до иконки, но фрейм у заголовка кривой.
Проявите лоялити к студенту
*/
}
}
1 change: 0 additions & 1 deletion frontend/src/blocks/profile/profile.css
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
display: flex;
align-items: center;
padding: 36px 0;
/*Если подгонять под ПЭПЭ, то тут padding: 40px 0; Проявите лоялити*/
}

@media screen and (max-width: 480px) {
Expand Down
2 changes: 0 additions & 2 deletions frontend/src/components/App.js
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,6 @@ function App() {
<div className="page__content">
<Header email={email} onSignOut={onSignOut} />
<Switch>
{/*Роут / защищён HOC-компонентом ProtectedRoute*/}
<ProtectedRoute
exact
path="/"
Expand All @@ -197,7 +196,6 @@ function App() {
onCardDelete={handleCardDelete}
loggedIn={isLoggedIn}
/>
{/*Роут /signup и /signin не является защищёнными, т.е оборачивать их в HOC ProtectedRoute не нужно.*/}
<Route path="/signup">
<Register onRegister={onRegister} />
</Route>
Expand Down

0 comments on commit 474ff67

Please sign in to comment.