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

Добавляет кнопки перемотки на 10 секунд #325

Merged
merged 2 commits into from
Sep 5, 2023

Conversation

Kreash
Copy link
Contributor

@Kreash Kreash commented May 31, 2023

Fix #324

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

Chrome:
image

Firefox:
image

@monochromer
Copy link
Member

Можно развить идею чуть дальше и добавить перемотку вперёд.

У встроенных элементов audio уже есть подобная функциональность: когда фокус установлен на элемент и при нажатии кнопок-стрелок "влево" и "вправо" Chrome перематывает на 1 минуту, Firefox - на 5 секунд, у Safari есть кнопки перемотки на 15 секунд. Можно переопределить и унифицировать это поведение.

src/pages/episode.njk Outdated Show resolved Hide resolved
@pepelsbey
Copy link
Member

players

Я немного порефакторил:

  • Добавил кнопку вперёд
  • Перерисовал иконки на симметричные
  • Вынес плеер в отдельный блок
  • Убрал проверку на недолёт/перелёт (браузеры действительно справляются сами)

Смешно, что в Safari эти кнопки уже есть, но думаю ничего страшного.

Что скажете? @Kreash @monochromer

@pepelsbey pepelsbey changed the title Добавляет кнопку перемотки на 10 секунд назад Добавляет кнопки перемотки на 10 секунд Sep 5, 2023
Copy link
Member

@monochromer monochromer left a comment

Choose a reason for hiding this comment

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

👍

@pepelsbey pepelsbey merged commit 6f99623 into web-standards-ru:main Sep 5, 2023
5 checks passed
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.

Добавить кнопку перемотки назад
3 participants