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

Add experimental URL timestamp functionality #214

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add experimental URL timestamp functionality #214

wants to merge 2 commits into from

Conversation

adfslslddkds
Copy link
Contributor

!!! Не коммитить в мастер !!!

К предложенному в #212

Реализовано:

  • Доступ по ссылке например переход по https://radio-t.com/p/2021/02/27/podcast-743?t=01:02:03 перейдет на страницу подкаста при нажатии плей плеер автоматически перемотает на выбранное место.
  • В меню воспроизведения добавлена кнопка дать ссылку. При нажатии копирует ссылку на текущее место воспроизведения в буфер обмена если возможно и вставляет в строку адреса если возможно. Кнопка криво отображается на мобилках, но пока что так.
  • При любом нажатии на паузу или на выбор новости по ссылке адрес добавляется в строку адреса. Оказалось не так уж и удобно переходить по истории браузера можно, но приходится каждый раз нажимать кнопку плей. Так как автовоспроизведение браузер блокирует. Еще отловил фичу браузера что при слишком частом добавлении ссылок в строку адреса и соответственно в историю браузера, браузер блокирует эту функциональность как слишком частый запрос. Для пользователя это выглядит как просто ничего не работает. Про блокировку можно узнать только в консоли браузера.

Нужна обратная связь =)

adfslslddkds and others added 2 commits March 4, 2021 20:57
Add fallbackCopyTextToClipboard function for copyTextToClipboard.
@adfslslddkds adfslslddkds marked this pull request as ready for review March 5, 2021 18:30
@akellbl4
Copy link
Collaborator

В целом идея классная, решение нужно еще допиливать. Пока не стал оставлять комментарии по коду потому что кажется что еще не готово.

@icegood
Copy link

icegood commented Apr 18, 2022

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

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