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

Embedded YouTube videos cannot start at a certain time using "?start=..." #328

Open
mjbrisebois opened this issue Jun 22, 2020 · 0 comments · May be fixed by #364
Open

Embedded YouTube videos cannot start at a certain time using "?start=..." #328

mjbrisebois opened this issue Jun 22, 2020 · 0 comments · May be fixed by #364

Comments

@mjbrisebois
Copy link

@mjbrisebois mjbrisebois commented Jun 22, 2020

Describe the bug

This code https://github.com/writeas/writefreely/blob/develop/postrender.go#L100-L102 (which is states the intention is to remove autoplay) makes it so that I cannot have YouTube videos start at a precise moment using ?start=....

Expected behavior

It should just remove the autoplay query parameter if that is the intention. Although I don't understand why that would happen on the blog page. It seems like something you would only want to remove on blog previews when there could be many annoying "autoplay"s triggered.

Application configuration

  • Multi-user mode
  • Database? sqlite
  • Open registration? yes
  • Federation enabled? no

Version or last commit: 4570511

Obayanju added a commit to Obayanju/writefreely that referenced this issue Aug 7, 2020
This uses go's html and url parser plus regex, instead of
using only a single regex for simplicity sake. A single regex expression
might be error prone, for example, when trying to matching html entities.

Fixes writeas#328
@Obayanju Obayanju linked a pull request that will close this issue Aug 7, 2020
1 of 1 task complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

2 participants
You can’t perform that action at this time.