Skip to content

Conversation

richarddushime
Copy link
Contributor

@richarddushime richarddushime commented Mar 9, 2025

I created a pop up form and created a button in the right corner,
the popup form i created an event listener to track if the link is clicked if yes the pop up will not show again and the button will remain , because a person may click the link but still does not fill the survey so there is a button for a backup

another thing is if the survey is not filled on each load/refresh the pop up will keep coming.

issue #243

@richarddushime richarddushime requested a review from a team as a code owner March 9, 2025 11:13
Copy link
Contributor

github-actions bot commented Mar 9, 2025

👍 All image files/references (if any) are in webp format, in line with our policy.

@LukasWallrich
Copy link
Contributor

Thanks, @richarddushime. The text is a little odd - as it sounds like people have already said that they will contribute to the survey. Maybe it rather needs to be "Help us improve the FORRT website" - "We would be grateful if you could complete this survey ..."? Also, can we avoid that it pops up repeatedly? For people who visit the webpage often, this will get annoying - but I don't know how easy it is to set a cookie that only shows this once a week when a user has not clicked on the survey, and never again when they have?

@richarddushime
Copy link
Contributor Author

richarddushime commented Mar 11, 2025

I have just made a commit

Currently the pop up will
Show immediately on first visit

Reappear after 7 days if not completed

Never show again once the survey is taken (This does not guarantee that a person filled the survey but i am considering a click)

for the button, it will appear but once clicked it will desapear

@LukasWallrich
Copy link
Contributor

@richarddushime I could now preview this in staging and discuss with Flavio - and we think it might be a bit too much ... especially because the website gets many new visitors who won't have feedback to share before they even see the website. The floating box looks great (though it would need to say something about a survey).

So, can we either set the popup to show up on the 5th visit, if we can increment cookies? Or remove it and just keep the beautiful floating box (with the tex mentioning the survey)? Thanks!

@richarddushime
Copy link
Contributor Author

richarddushime commented Apr 14, 2025

@LukasWallrich @flavioazevedo

I m Sorry for taking so long to address this .

I did set the survey pop up form display on the 5th visit as suggested when the survey is completed it will no longer display
Note: the completion is considered by the user click , As i mentionned in the Description , and the button remains

Additionnaly , What would you want to see on the btn ? I think personnally we do not need to add a lot on the button, because once the user opens the survey there is more informations before filling the form.

For testing

Use a Incognito or a different browser from what you are using if you ever clicked on the survey link,
count to 5 times new tabs or open and close or refresh 5 times
After 5 time if you dont fill or click the link , it will keep showing

Copy link
Contributor

This pull request has been inactive for more than 90 days. If there is no further activity, it will be automatically closed in seven days time. You can reopen the pull request if it is still relevant.

@github-actions github-actions bot added the stale Marked by labeler label Jun 14, 2025
Copy link
Contributor

This pull request has been automatically closed due it being stale for more than 7 days. Please feel free to reopen if you still want to submit this pull request.

@github-actions github-actions bot closed this Jun 21, 2025
@github-actions github-actions bot removed the stale Marked by labeler label Jun 22, 2025
@richarddushime richarddushime changed the title POP UP survey form POP UP survey form and button Aug 4, 2025
@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

2 similar comments
@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

29 similar comments
@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

@LukasWallrich
Copy link
Contributor

✅ This PR has been deployed to staging as part of an aggregated deployment. View at: https://staging.forrt.org

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants