Ivan Bogachev

Расскажите о минусах и плюсах

Из плюсов - идея интересная, выглядит симпатично.
Из минусов (в самом посте):

• Шрифт мелкий, строки длинные. На 15-дюймовом FullHD - сложно читать. Я бы увеличил хотя бы до 16-18px.
  
• Контрастность тоже не самая лучшая. Серый текст по белому фону заставляет напрягать глаза.
  
• Вертикальный ритм немного странный. Может быть над заголовками добавить отступы?
  
• Картинки хочется сделать побольше. Хотя может быть и нет. Но отделить их отступами точно стоит.
  
• Не хватает информации об авторе, даты написания статьи (может все уже устарело на 10 лет).
  
• Иконки для расшаривания в социальных сетях тоже можно добавить.
  
• Так и хочется внизу увидеть теги, комментарии, похожие посты и.т.д. Их там нет, а очень хочется.
  
• Эффекты при наведении мыши и фокусе на элементы нужно добавить.
  

Скролл очень сильно лагает, так что навигацию оценить сложно. Адаптивность отсутствует. Определенно стоит подумать про сценарии взаимодействия с сайтом в целом. Если это - блог, то там точно должно быть (люди к этому привыкли и ждут этого):

• Список статей, который можно быстро пролистывать.
  
• Поиск по статьям. Если материалов мало, то по крайней мере поиск по тегам или категориям.
  
• Страницы авторов, или по крайней мере хоть какая-то о них информация, контакты.
  

и.т.д.

Красивые анимации - это уже вторичное, начинать нужно со сценариев взаимодействия, а то получается красивое, но не удобное.

По всей видимости вам подойдет свойство object-fit. Для старых браузеров понадобится полифилл, например этот.

Добавлю еще пару слов про эту задачу в целом (не про сайт по ссылке в вопросе, а вообще). Если нужно делать такую статичную штуку с выбором цвета стен, обоев, полов и.т.д., особенно если нужно дать пользователю возможность выбирать свой вариант, то можно поиспользовать SVG. Это немного внезапно, но тем не менее. Основная часть комнаты, тени и рефлексы - это обычная картинка с прозрачностью в нужных местах (ее можно встроить в SVG через тег image), стены, полы - это простые прямоугольники, на которые наложены 3D-трансформации. Дальше их можно заливать как угодно - цветами, текстурами... и не нужно будет ломать себе мозг вопросом о том, как на лету повернуть "нормальную" текстуру и сделать из нее пол с перспективой. Мебель заливается цветом не намного сложнее - делается SVG-маска в форме предмета мебели и применяется фильтр hue-rotate. Таким образом можно "на коленке" соорудить работающий вариант такой презентации.

На сколько так правильно делать?

Мораль - понятие растяжимое. Вроде бы хочется сказать, что "низя-низя так делять", но с другой стороны не сходив так на несколько собеседований не получишь понимание того, кто ты есть на рынке, что вообще от тебя хотят и что могут дать. Вряд ли есть единственно-правильный ответ на этот вопрос. Тут как бы конфликт интересов работодателей и работников.

Учитывая тот факт, что (в жизни всякое бывает) может я захочу в будущем вернуться к данной вакансии.

По моему опыту если разойтись с работодателем спокойно, до оформления каких-либо бумаг, то через некоторое время вполне можно вновь встретиться. У меня есть несколько компаний, с которыми мы где-то раз в год пересекаемся, не подходим друг другу и сохраняем контакт на будущее. Главное тут - наличие взаимного уважения. Всякие казусы случаются в жизни и отказаться можно по тысяче причин. А попросить указать на свои ошибки, как и предоставить свой отзыв - это неплохая мысль. Особенно если собеседующий вас человек был адекватен. У меня даже было такое, что молодой владелец небольшой компании поблагодарил за предоставленный отзыв и обмен опытом в процессе общения. До этого случая я думал, что такого вообще не бывает, а он восстановил мою веру в человечество. Так что смотрите сами, никто вас ругать ни за что не будет.

P.S.: А по поводу "домашних заданий" в комментариях верно говорят, зачастую их пачками раздают и вероятность положительного исхода стремится к нулю.

Что посоветуете? Брать тестера? Менять разработчиков? Менять процессы?

На мой взгляд (личный опыт, могу ошибаться) полезно поговорить с разработчиками и объяснить им, что есть задачи бизнеса. Они не в вакууме код пишут. Есть пользователи программного продукта, которым нужно дать возможность решать какие-то свои задачи с его помощью. Это - их главная работа, а не формальное выполнение тасков из джиры. Пока мышление у них не перестроится вы можете нанимать тестеров, менять процессы, но подход "и так сойдет" не искоренится.

А как придет понимание, что есть пользователи и нужно о них заботиться, уже можно говорить о внедрении планок качества или об автоматизации тестирования. Можно начать не с хардкорного TDD, а с более мягкого подхода, соответствующего этой идее: взять что-нибудь вроде codecept.js и по-быстрому описывать сценарии взаимодействия пользователя с системой. Сначала основные, которые обязательно должны работать, потом альтернативные, где обработка ошибок происходит. Собственно и изначальное ТЗ формулировать в виде сценария взаимодействия. Можно отвести под тесты отдельный сервер (ненужное железо всегда можно где-нибудь найти). Потом, как привыкнут, попробовать покрывать все юнит тестами или еще что-то внедрять, своим примером показывая, что это и правда может экономить время на поиск багов или давать какие-то еще полезности. Иными словами стоит сначала перестраивать мышление, а потом уже улучшать технические моменты.

Не любитель реакта, поэтому про него не буду говорить. А вот CSS покритикую:
- Стоит прикрутить какой-нибудь препроцессор, поиспользовать вложенность (структура лучше будет видна) и вынести в человеко-понятные константы все, что выносится - цвета, размеры и.т.д. Там достаточно повторений.
- Стоит поделить все на отдельные файлы-компоненты.
- Стоит получше подумать над общим разбиением CSS на компоненты. Есть конечно разные подходы, но отдельные кнопки, или группа из нескольких кнопок, или чекбоксы - это универсальные штуки на весь проект. Какой смысл их привязывать к какому-то сайдбару или калькулятору?
- Про адаптивность вы сами написали.
- Стили для :focus отсутствуют. Клавиатурой не получится пользоваться.
- Еще мне кажется, что у сайдбара отступ внизу должен быть (дизайн не видел, но имхо есть). И что cursor: pointer у кнопок должен быть.

З.Ы.: Еще есть мысль, что вариант "все" там не нужен. "Все" должны показываться при отсутствии фильтров. Но без анализа ЦА не буду утверждать, может там к к такому варианту люди привыкли.