Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implemented a User-Friendly DOM-Based Message System to Replace Intrusive Alert Messages #918

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

barchakuz
Copy link
Contributor

Summary

I replaced the pages' alert message system with a more user-friendly DOM-based message system. Previously, the alert messages were disruptive and required users to close them before interacting with the page. The new system displays messages within the DOM and automatically disappears after 3 seconds. This change aligns with HCI standards by reducing user effort for minor tasks and improving overall user experience.

Motivation

The motivation behind this Pull Request is to enhance user experience by addressing the inconvenience caused by the previous alert message system. The alerts were intrusive and hindered user interaction with the page. By implementing a DOM-based message system with automatic dismissal, users can now seamlessly continue their tasks without interruption.

Testing

I extensively tested the new message system to ensure its functionality and usability. Beyond the automatic hooks, I manually verified that messages appear correctly, disappear after 3 seconds, and do not interfere with user interaction. Additionally, I tested the system across different browsers and devices to ensure compatibility.
##Screenshot
NOTE: In both videos i tried uploading wrong file instead to Zip to show error, file upload was not capture by screen recorder but message is visible
Before

2024-03-30.02-14-59.mp4

After

2024-03-30.02-14-15.mp4

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.

None yet

1 participant