feat: add Product-section, Team-section, Process-section and fix comm… - #2
feat: add Product-section, Team-section, Process-section and fix comm…#2oostap1985 wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Не очевидное название файла, надо более понятное
| @@ -0,0 +1,14 @@ | |||
| [ | |||
| { | |||
| "img": "open_source", | |||
There was a problem hiding this comment.
Разная нотация названия переменных. Лучше тогда уж везде kebab-case
| "img": "photo1", | ||
| "profession": "инженер программист, изобретатель", | ||
| "description": "Имеет профессиональный опыт более десяти лет. Работал фронтенд и бэкенд-разработчиком, а также занимался инфраструктурой и внедрением инженерных практик. Создатель open source фреймворка mlut, аналога Tailwind для вёрстки кастомных сайтов и креативов.", | ||
| "connection": "yes" |
There was a problem hiding this comment.
Это ведь boolean, зачем строка
| "name": "Олег Остапчук", | ||
| "img": "photo4", | ||
| "profession": "Разработчик", | ||
| "description": "Имеет профессиональный опыт более десяти лет. Работал фронтенд и бэкенд-разработчиком, а также занимался инфраструктурой и внедрением инженерных практик. Создатель open source фреймворка mlut, аналога Tailwind для вёрстки кастомных сайтов и креативов.", |
There was a problem hiding this comment.
Про себя тоже релевантный текст напиши, потом отредактируем
| @@ -0,0 +1,11 @@ | |||
| <div class="D-f Fld-c Gap2u"> | |||
| <div class="Bd1;s;$accent100 Bdrd2u Ov-h W242 md_W326 H144 md_H228"> | |||
There was a problem hiding this comment.
Mnh, вместо H и размеры лучше в u - pixel perfect не нужен
|
|
||
| <profile-card class="D-f Fld-c Ai-c Jc-sb Gap3u no-js W242 Mnh375 lg_W305 lg_Mnh390 xxl_W350"> | ||
| <div class="Ps D-f Fld-c Gap1u"> | ||
| <div class="Bdrd100p Ov-h Ojf Ojp-c -Sz100 lg_-Sz140"> |
There was a problem hiding this comment.
object-fit разве не на сам img вешается?
There was a problem hiding this comment.
На CSS нет вариантов сделать эту анимацию?
| const btnText = this.button.textContent; | ||
| const newText = btnText === 'Свернуть' ? 'Подробнее' : 'Свернуть'; | ||
| this.button.textContent = newText; | ||
| this.classList.toggle('no-js'); |
There was a problem hiding this comment.
Почему no-js снимается при переключении, а не при инициализации компонента?
| %> | ||
|
|
||
| <%# Process-section %> | ||
| <process-scroll class="D-f Fld-c Gap2.5u W100p P5u;4u md_Gap10u md_P0;0;0;15u xl_P0;0;0;20u xxl_P0;0;0;25u"> |
There was a problem hiding this comment.
Почему здесь однотипный код не в цикле?
There was a problem hiding this comment.
Сейчас композиция заканчивается обрубком. Лучше, чтобы хотя бы хвост был, как перед концом пункта
There was a problem hiding this comment.
И на этом этапе не требовалось делать анимацию линий, только скролл. Такое надо на webgl, по идее, потому что на CSS может тормозить
| @@ -1,38 +1,30 @@ | |||
| <% | |||
| const css = { | |||
| const stylesCss = { | |||
There was a problem hiding this comment.
Все еще не исправлено. Если словарь заканчивается на css - значит это словарь алиасов. А тут совершенно разные по смыслу значения лежат
There was a problem hiding this comment.
На мобильном в hero секции нет смысла декоратвное изображение показывать
There was a problem hiding this comment.
На квадратном мониторе, в секции "услуги" лучше 2 равные колоники, а то сейчас странно выглядит, что одна шире
| </ul> | ||
| </section> | ||
|
|
||
| <%# Product-section %> |
There was a problem hiding this comment.
На десктопе не соответствует макету
2 participants
Привет.
Это еще не конечный результат. Посмотри, пожалуйста, на анимацию в Process секции, это примерно то, что вы хотели?