Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Develop #36

Merged
merged 477 commits into from
Nov 18, 2020
Merged

Develop #36

merged 477 commits into from
Nov 18, 2020

Conversation

OctopussyO
Copy link
Contributor

@OctopussyO OctopussyO commented Nov 4, 2020

Ссылка на gh-pages

Верстка отзывчивая. Реализована основная функциональность.

Из легализованного дополнительного:

  • есть возможность перемещать ползунок плеера (onClick, onMove),
  • при смене треков меняется фон приложения,
  • плавность: анимированы ховеры для ссылок, раскрытие плеера, выпадающий ссылок на маленьких разрешениях разрешениях, плавная смена фона при переключении треков,
  • если название исполняемого трека не помещается в контейнер, "появляется" бегущая строка (в контейнере с треками при переполнении просто троеточие, мы посчитали, что иначе очень много подвижных элементов),
  • бегущая строка работает и при ресайзе и при раскрытии плеера,
  • есть кастомная валидация формы,
  • добавлено предупреждение о том, что необходимо заполнить все поля формы,
  • есть визуализация аудио для десктопов и планшетов. Для мобильных устройств визуализацию отключаем, т.к. на айфонах всё-таки дергается звук. На планшетах оставили, звук ровный, но может подрагивать, если агрессивно перематывать сразу после запуска песни (аудио не успевает полностью загружаться).

Так же есть заготовка API для отправки формы и получения информации о треках.

Из отсебятины:

  • добавлено свечение на кнопки при наведении,
  • добавлено размытие границ контейнеров с текущим треком и со списком треков,
  • добавлены прелоадеры на загрузку названия и кавера трека, в случае отсутствия у трека обложки загружается заглушка,
  • добавлены медиазапросы для ландшафтной ориентации,
  • добавлено лёгкое затемнение фона при открытом состоянии плеера на малых разрешениях для лучшей читаемости текста,
  • добавлен несколько отличающийся от макета ховер на кнопку отправки формы (в качестве предложения, наклон текста был мало заметен)?
  • добавлен favicon.
    Ещё мы сделали вот такую визуализацию, но немного не успели её оптимизировать (на мобильных устройствах начинает сбиваться немного звук), она сейчас лежит в отдельной ветке, выглядит вот так.

Сейчас треки лежат файлами в папке src/db/audio
Цвета фона сделали градиентами, потому что даже при оптимизированных картинках приложение очень долго загружалось.
Не успели сделать изменение цвета текста при смене треков, но для этого почти всё готово (поле с цветом в объекте трека есть, иконки сделаны svg-компонентами).

Copy link
Collaborator

@spacehaz spacehaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

мне нравится как реализовано, единственное (в том числе с точки зрения UX) меня немного смутило пара моментов:

  1. почему то при закрытии/открытии плеера ползунок меняет свое положение (понимаю что не доделано еще, просто обратите внимание)
  2. плеер закрывается моментально, а открывается с задержкой. если это задумка, то оставляйте) просто обратите тоже на это внимание
  3. откажитесь от браузерных формулировок ошибок, они должны быть кастомными и человекочитаемыми, так как сайтом будут пользоваться русскоязычные пользователи без достаточного понимания формулировок которые есть на данный момент. Для этого смотрите в сторону содержимого объекта validity

src/components/App.js Show resolved Hide resolved
src/components/Footer.css Outdated Show resolved Hide resolved
src/components/Form.css Outdated Show resolved Hide resolved
src/components/Form.js Outdated Show resolved Hide resolved
src/components/Form.js Outdated Show resolved Hide resolved
src/components/ProjectLogo.js Show resolved Hide resolved
src/components/ServiceLinks.js Outdated Show resolved Hide resolved
src/components/TextContainer.js Outdated Show resolved Hide resolved
src/components/TextContainer.js Outdated Show resolved Hide resolved
src/components/App.css Show resolved Hide resolved
dariaovch and others added 29 commits November 17, 2020 16:46
refactor: SVG - reviewer's comment
fix: header links styles for small-height smartphones
Fix: fixed margin-bottom for last textContainer
@OlyErofeeva OlyErofeeva merged commit 766413b into main Nov 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants