Skip to content

Switch off autoplay in 2.7.0-rc1 post #2318

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

Merged
merged 1 commit into from
Dec 19, 2019
Merged

Switch off autoplay in 2.7.0-rc1 post #2318

merged 1 commit into from
Dec 19, 2019

Conversation

stomar
Copy link
Contributor

@stomar stomar commented Dec 19, 2019

It's more annoying than helping. At the time when one reaches the paragraph, the video has been playing for some time already, and one needs to stop and rewind. It's better to simply start the video at the time one wants to see it.

@stomar stomar requested review from a team as code owners December 19, 2019 21:00
@stomar stomar merged commit 727bfc6 into ruby:master Dec 19, 2019
@stomar stomar deleted the autoplay branch December 19, 2019 21:07
stomar added a commit that referenced this pull request Dec 23, 2019
@stomar
Copy link
Contributor Author

stomar commented Dec 23, 2019

@nurse Some feedback would be welcome, especially in regard to the upcoming final release.

@ismailarilik
Copy link
Member

I think it is better to make this kind of language-agnostic changes to all translations since it is faster to make them on one branch than making them on different branchs via different PRs.

@ismailarilik
Copy link
Member

Sorry about the comment above, I have thought that there was corresponding Turkish translation but I have realized that it wasn't merged yet.

@nurse
Copy link
Member

nurse commented Dec 25, 2019

I think it should be autoplay because the video is just a decoration and readers feel the experience.

stomar added a commit that referenced this pull request Dec 25, 2019
@stomar
Copy link
Contributor Author

stomar commented Dec 25, 2019

ACK; fixed.

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