Skip to content

Open landing page video in new tab with confirmation dialog

David requested to merge start-video into master

Closes #617 (closed)

What does this MR do?

Instead of embedding the video into the page, clicking on the video preview open the video in a new tab.

To inform users that they are leaving our page and are redirect to youtube a confirmation dialog is shown before opening the new tab:

Bildschirmfoto_2019-11-12_um_20.08.14

How confident are you it won't break things if deployed?

Very

Links to related issues

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s))
  • Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings

Merge request reports