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
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | Screen Shot 2016-08-13 at 9.07.22 AM.png | 178.03 KB | jrockowitz |
| #6 | replace_embedded_videos-2783361-6.patch | 21.17 KB | jrockowitz |
Comments
Comment #2
jrockowitz commentedComment #6
jrockowitz commentedAttached 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.
Comment #10
jrockowitz commentedComment #12
ifrikThanks a lot!