Problem/Motivation

The documentation mainly consists out of youtube videos that are embedded in the help page and in most admin pages. These videos are certainly helpful but they create some problems: mainly the lack of translation and third-party tracking/cookies.

Translation: Videos are difficult to translate so help and documentation are almost exclusively in English.

Tracking and cookies: Due to the embedded videos, tracking and cookies come from google.com, youtube.com, ytimg.com, and doubleclick.com. This makes the module difficult to use for any site that needs to deal with privacy issues.
In countries with legal rules about placing cookies (such as for example the Netherlands and other EU countries), users needs to give permission/consent for the placement of cookies that are not essential to the functioning of the site. For example, if a user with the permissions to see the results of a form, goes to the relevant admin page, five sites appear to be tracking the user, and youtube places cookies.
So, legally the site would need to get a permission by users before they can access the admin page with the results - independent on whether they are even planning to watch the video on that page or not. If permission is not given, then either the access would need to be denied, or a version of the admin page without the embedded video would need to be displayed.

Proposed resolution

Replacing the embedded videos with a link to the videos on youtube would solve the whole problem of tracking and cookies and would also improve page load time.

Additionally, writing up some of the documentation/help provided would allow the text to be translated - but that's probably another issue.

Remaining tasks

User interface changes

This is a UI change.

API changes

Data model changes

Comments

ifrik created an issue. See original summary.

jrockowitz’s picture

Assigned: Unassigned » jrockowitz

  • jrockowitz committed 23bedd0 on 2783361-link-videos
    Issue #2783361: Replace embedded videos by links to them.
    

  • jrockowitz committed aea7207 on 2783361-link-videos
    Issue #2783361: Replace embedded videos by links to them.
    

  • jrockowitz committed 05ea76d on 2783361-link-videos
    Issue #2783361: Replace embedded videos by links to them.
    
jrockowitz’s picture

Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new21.17 KB
new178.03 KB

Attached is the patch.

Below is a screen shot of the new admin setting (/admin/structure/yamlform/setting)

@ifrik Since the change is fairly simple. If all the tests pass I am just going to apply the patch and mark this ticket as fixed.

  • jrockowitz committed b1d433b on 2783361-link-videos
    Issue #2783361: Replace embedded videos by links to them. Fix breadcrumb...

  • jrockowitz committed a1f1533 on 2783361-link-videos
    Issue #2783361: Replace embedded videos by links to them. Fix breadcrumb...

  • jrockowitz committed 8049242 on 8.x-1.x
    Issue #2783361 by jrockowitz: Replace embedded videos by links to them
    
jrockowitz’s picture

Status: Needs review » Fixed

  • jrockowitz committed 7732947 on 8.x-1.x
    Issue #2783361: Replace embedded videos by links to them.
    
ifrik’s picture

Thanks a lot!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.