-
Notifications
You must be signed in to change notification settings - Fork 3
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
[Feature] Add Toast Notifications #1
base: main
Are you sure you want to change the base?
Conversation
added basic toasts/notifications to settings.jsp to acknowledge uploaded data is processing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔥
// Manually submit the form after a short delay to allow the toast to show | ||
setTimeout(() => { | ||
form.submit(); | ||
}, 2000); // 2 seconds |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd lower this to maybe 500, or remove entirely.
<script> | ||
document.addEventListener("DOMContentLoaded", function() { | ||
const rpcForm = document.querySelector("form[action='/settings']"); | ||
const privateKeyForm = document.querySelector("form[action='/privateKeyPost']"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could do the same with the privateKeyUpload (upload file) selector.
Left minor comments. |
Will address minor comments and make the appropriate fixes. |
The goal of this PR is to introduce basic toasts/notifications to the Arcana settings page
When a user uploads an RPC URL or a private key correctly, a toast will appear to acknowledge the information is processing.
Additional goals: