diff --git a/404.html b/404.html index fa4e220c8..e7ef4d0cc 100644 --- a/404.html +++ b/404.html @@ -1,6 +1,7 @@ +
diff --git a/authError.html b/authError.html index 1c3b27b45..19954d9f7 100644 --- a/authError.html +++ b/authError.html @@ -1,6 +1,7 @@ +
Sorry! You are not allowed to be here :(
diff --git a/close.html b/close.html index ec4302df9..bc7226b80 100644 --- a/close.html +++ b/close.html @@ -1,8 +1,31 @@ + + + - -
Thank you for your feedback!
+ +
+
+

Thank you for your feedback!

+
+
+

+ Did you know? + +

+
+
+
- \ No newline at end of file + diff --git a/close2.html b/close2.html new file mode 100644 index 000000000..d9f267d3d --- /dev/null +++ b/close2.html @@ -0,0 +1,33 @@ + + + + + + + + +
+
+

+
+
+

+ Did you know? + +

+
+
+
+ + diff --git a/conference.js b/conference.js index b10f6c556..b91fba106 100644 --- a/conference.js +++ b/conference.js @@ -184,29 +184,34 @@ function muteLocalVideo (muted) { * If requested show a thank you dialog before that. * If we have a close page enabled, redirect to it without * showing any other dialog. - * @param {boolean} showThankYou whether we should show a thank you dialog + * + * @param {object} options Feedback data + * @param {boolean} options.thankYouDialogVisible - whether we should + * show thank you dialog + * @param {boolean} options.feedbackSubmitted - whether feedback was submitted */ -function maybeRedirectToWelcomePage(showThankYou) { - +function maybeRedirectToWelcomePage(options) { // if close page is enabled redirect to it, without further action if (config.enableClosePage) { - window.location.pathname = "close.html"; + if (options.feedbackSubmitted) + window.location.pathname = "close.html"; + else + window.location.pathname = "close2.html"; return; } - if (showThankYou) { + // else: show thankYou dialog only if there is no feedback + if (options.thankYouDialogVisible) APP.UI.messageHandler.openMessageDialog( null, "dialog.thankYou", {appName:interfaceConfig.APP_NAME}); - } - if (!config.enableWelcomePage) { - return; + // if Welcome page is enabled redirect to welcome page after 3 sec. + if (config.enableWelcomePage) { + setTimeout(() => { + APP.settings.setWelcomePageEnabled(true); + window.location.pathname = "/"; + }, 3000); } - // redirect to welcome page - setTimeout(() => { - APP.settings.setWelcomePageEnabled(true); - window.location.pathname = "/"; - }, 3000); } /** diff --git a/css/_redirect_page.scss b/css/_redirect_page.scss index 5408762da..af73e4150 100644 --- a/css/_redirect_page.scss +++ b/css/_redirect_page.scss @@ -6,7 +6,36 @@ html, body { } .redirectPageMessage { + width: 30%; + margin: 20% auto; text-align: center; - font-size: 36px; - margin-top: 20%; + font-size: 24px; + + .thanks-msg { + border-bottom: 1px solid $selectBg; + padding-left: 30px; + padding-right: 30px; + p { + margin: 30px auto; + font-size: 24px; + line-height: 24px; + } + } + .hint-msg{ + p { + margin: 26px auto; + font-weight: 600; + font-size: 16px; + line-height: 18px; + .hint-msg__holder{ + font-weight: 200; + } + } + .happy-software{ + width: 120px; + height: 86px; + margin: 0 auto; + background: $happySoftwareBackground; + } + } } diff --git a/css/_variables.scss b/css/_variables.scss index 421497793..8d1d5d517 100644 --- a/css/_variables.scss +++ b/css/_variables.scss @@ -92,6 +92,8 @@ $borderRadius: 4px; $defaultWatermarkLink: '../images/watermark.png'; $sidebarWidth: 200px; +$happySoftwareBackground: transparent; + /** * Z-indexes. TODO: Replace this by a function. */ diff --git a/modules/UI/UI.js b/modules/UI/UI.js index f4cb07285..1b90e865d 100644 --- a/modules/UI/UI.js +++ b/modules/UI/UI.js @@ -1037,25 +1037,26 @@ UI.requestFeedbackOnHangup = function () { if (Feedback.isVisible()) return Promise.reject(UIErrors.FEEDBACK_REQUEST_IN_PROGRESS); // Feedback has been submitted already. - else if (Feedback.isEnabled() && Feedback.isSubmitted()) - return Promise.resolve(true); + else if (Feedback.isEnabled() && Feedback.isSubmitted()) { + return Promise.resolve({ + thankYouDialogVisible : true, + feedbackSubmitted: true + }); + } else return new Promise(function (resolve) { if (Feedback.isEnabled()) { - // If the user has already entered feedback, we'll show the - // window and immidiately start the conference dispose timeout. - if (Feedback.getFeedbackScore() > 0) { - Feedback.openFeedbackWindow(); - resolve(false); - - } else { // Otherwise we'll wait for user's feedback. - Feedback.openFeedbackWindow(() => resolve(false)); - } + Feedback.openFeedbackWindow( + (options) => { + options.thankYouDialogVisible = false; + resolve(options); + }); } else { // If the feedback functionality isn't enabled we show a thank // you dialog. Signaling it (true), so the caller // of requestFeedback can act on it - resolve(true); + resolve( + {thankYouDialogVisible : true, feedbackSubmitted: false}); } }); }; diff --git a/modules/UI/feedback/FeedbackWindow.js b/modules/UI/feedback/FeedbackWindow.js index f6254557e..1bab72a0b 100644 --- a/modules/UI/feedback/FeedbackWindow.js +++ b/modules/UI/feedback/FeedbackWindow.js @@ -164,9 +164,7 @@ export default class Dialog { } setFeedbackMessage() { - let message = $('#feedbackTextArea').val(); - - this.feedbackMessage = message; + this.feedbackMessage = $('#feedbackTextArea').val(); } show(cb) { @@ -179,6 +177,8 @@ export default class Dialog { } onHide() { - this.onCloseCallback(this.feedbackScore, this.feedbackMessage); + this.onCloseCallback({ + feedbackSubmitted: this.submitted + }); } }