From 96582bf426541fca5dd23e5dce0c417149656797 Mon Sep 17 00:00:00 2001 From: Srikar-1729 Date: Sun, 20 Oct 2024 18:58:43 +0530 Subject: [PATCH 1/4] Feedback form emoji not selected msg added in ui --- assets/js/script.js | 9 ++++++--- index.html | 2 +- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/assets/js/script.js b/assets/js/script.js index 8ac10063..d3b4fa82 100644 --- a/assets/js/script.js +++ b/assets/js/script.js @@ -92,10 +92,11 @@ document.getElementById('closeModal').onclick = function() { document.getElementById('nextToFeedback').onclick = function() { const selectedEmoji = document.querySelector('.emoji.selected'); - if (!selectedEmoji) { - alert('Please select an emotion!'); + if (!selectedEmoji) { + document.getElementsByClassName('emojis')[0].style.margin="0 0 20px 0"; + document.getElementById('emoji-not-selected').removeAttribute("hidden"); return; - } + } selectedEmotion = selectedEmoji.dataset.value; document.getElementById('step1').style.display = 'none'; document.getElementById('step2').style.display = 'block'; @@ -105,6 +106,8 @@ document.querySelectorAll('.emoji').forEach(emoji => { emoji.onclick = function() { document.querySelectorAll('.emoji').forEach(e => e.classList.remove('selected')); emoji.classList.add('selected'); + document.getElementById('emoji-not-selected').hidden= true; + document.getElementsByClassName('emojis')[0].style.margin="20px 0"; } }); diff --git a/index.html b/index.html index 9a1b75ee..9128ebbf 100644 --- a/index.html +++ b/index.html @@ -223,7 +223,7 @@

We Value Your Feedback

How do you feel?

- +
😄 From 8768e41b1a7144a6fdfe76a261b23e7e3d396d6a Mon Sep 17 00:00:00 2001 From: Srikar-1729 Date: Sun, 20 Oct 2024 19:32:55 +0530 Subject: [PATCH 2/4] Trigger deployment --- assets/js/script.js | 1 + 1 file changed, 1 insertion(+) diff --git a/assets/js/script.js b/assets/js/script.js index d3b4fa82..f7b9cbdb 100644 --- a/assets/js/script.js +++ b/assets/js/script.js @@ -109,6 +109,7 @@ document.querySelectorAll('.emoji').forEach(emoji => { document.getElementById('emoji-not-selected').hidden= true; document.getElementsByClassName('emojis')[0].style.margin="20px 0"; } + }); document.getElementById('nextToEmail').onclick = function() { From 835fafeda0924f9bfa5e4f4d6f580129bbc4be4c Mon Sep 17 00:00:00 2001 From: Srikar-1729 Date: Sun, 20 Oct 2024 19:59:12 +0530 Subject: [PATCH 3/4] Retrying deployment --- assets/js/script.js | 1 - 1 file changed, 1 deletion(-) diff --git a/assets/js/script.js b/assets/js/script.js index f7b9cbdb..d3b4fa82 100644 --- a/assets/js/script.js +++ b/assets/js/script.js @@ -109,7 +109,6 @@ document.querySelectorAll('.emoji').forEach(emoji => { document.getElementById('emoji-not-selected').hidden= true; document.getElementsByClassName('emojis')[0].style.margin="20px 0"; } - }); document.getElementById('nextToEmail').onclick = function() { From 1a6506d5fd0fefaa63c43817c2883fba9707482c Mon Sep 17 00:00:00 2001 From: Srikar-1729 Date: Sun, 20 Oct 2024 20:11:07 +0530 Subject: [PATCH 4/4] Redeploy --- assets/js/script.js | 1 + 1 file changed, 1 insertion(+) diff --git a/assets/js/script.js b/assets/js/script.js index d3b4fa82..723fdc32 100644 --- a/assets/js/script.js +++ b/assets/js/script.js @@ -108,6 +108,7 @@ document.querySelectorAll('.emoji').forEach(emoji => { emoji.classList.add('selected'); document.getElementById('emoji-not-selected').hidden= true; document.getElementsByClassName('emojis')[0].style.margin="20px 0"; + } });