From e1298e4fd96bae86af23194c6fba50ba3dca4ca4 Mon Sep 17 00:00:00 2001 From: Chi Vinh Le Date: Thu, 24 May 2018 19:00:07 +0200 Subject: [PATCH 1/4] Fix Touch issues on IOS Safari (iPad) --- views/embed/stream.njk | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/views/embed/stream.njk b/views/embed/stream.njk index ddc620f8e..d6c53b210 100644 --- a/views/embed/stream.njk +++ b/views/embed/stream.njk @@ -9,6 +9,13 @@ {% block html %}
+ +{# + Resolves touch handling issues encountered on IOS Safari under certain circumstances + https://www.pivotaltracker.com/n/projects/1863625 +#} + + {% endblock %} {% block js %} From 540df09b5bbe60af48ea4473e0beb750a7399ed9 Mon Sep 17 00:00:00 2001 From: Wyatt Johnson Date: Fri, 25 May 2018 11:48:55 -0600 Subject: [PATCH 2/4] Update stream.njk --- views/embed/stream.njk | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/views/embed/stream.njk b/views/embed/stream.njk index d6c53b210..8b9cf52b6 100644 --- a/views/embed/stream.njk +++ b/views/embed/stream.njk @@ -11,8 +11,8 @@
{# - Resolves touch handling issues encountered on IOS Safari under certain circumstances - https://www.pivotaltracker.com/n/projects/1863625 + Resolves touch handling issues encountered on IOS Safari under certain circumstances: + https://stackoverflow.com/questions/12363742/touchstart-event-is-not-firing-inside-iframe-ios-6 #} From c1eab3f2c65b46de00daa33f662ffc8322b2e91e Mon Sep 17 00:00:00 2001 From: Wyatt Johnson Date: Fri, 25 May 2018 11:51:43 -0600 Subject: [PATCH 3/4] moved event handler to bundle --- client/coral-embed-stream/src/index.js | 5 +++++ views/embed/stream.njk | 7 ------- 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/client/coral-embed-stream/src/index.js b/client/coral-embed-stream/src/index.js index a4c6bcd39..1142239c6 100644 --- a/client/coral-embed-stream/src/index.js +++ b/client/coral-embed-stream/src/index.js @@ -8,6 +8,11 @@ import reducers from './reducers'; import TalkProvider from 'coral-framework/components/TalkProvider'; import pluginsConfig from 'pluginsConfig'; +// Resolves touch handling issues encountered on IOS Safari under certain +// circumstances. It may be related to issues reported here: +// https://stackoverflow.com/questions/12363742/touchstart-event-is-not-firing-inside-iframe-ios-6 +document.body.addEventListener('touchstart', () => {}); + async function main() { const context = await createContext({ reducers, diff --git a/views/embed/stream.njk b/views/embed/stream.njk index 8b9cf52b6..ddc620f8e 100644 --- a/views/embed/stream.njk +++ b/views/embed/stream.njk @@ -9,13 +9,6 @@ {% block html %}
- -{# - Resolves touch handling issues encountered on IOS Safari under certain circumstances: - https://stackoverflow.com/questions/12363742/touchstart-event-is-not-firing-inside-iframe-ios-6 -#} - - {% endblock %} {% block js %} From ba2abab3589211aaf6ebec1854147f67ef892670 Mon Sep 17 00:00:00 2001 From: Wyatt Johnson Date: Fri, 25 May 2018 11:54:34 -0600 Subject: [PATCH 4/4] added pivotal tracker ref --- client/coral-embed-stream/src/index.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/client/coral-embed-stream/src/index.js b/client/coral-embed-stream/src/index.js index 1142239c6..a2718e49d 100644 --- a/client/coral-embed-stream/src/index.js +++ b/client/coral-embed-stream/src/index.js @@ -10,7 +10,10 @@ import pluginsConfig from 'pluginsConfig'; // Resolves touch handling issues encountered on IOS Safari under certain // circumstances. It may be related to issues reported here: -// https://stackoverflow.com/questions/12363742/touchstart-event-is-not-firing-inside-iframe-ios-6 +// +// https://stackoverflow.com/questions/12363742/touchstart-event-is-not-firing-inside-iframe-ios-6 +// +// Further details: https://www.pivotaltracker.com/story/show/157794038 document.body.addEventListener('touchstart', () => {}); async function main() {