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

spoor error messages: switch to using Notify for all error messages #245

Open
moloko opened this issue Dec 8, 2020 · 0 comments
Open

spoor error messages: switch to using Notify for all error messages #245

moloko opened this issue Dec 8, 2020 · 0 comments
Assignees

Comments

@moloko
Copy link
Contributor

moloko commented Dec 8, 2020

As suggested by @tomgreenfield here we might as well switch from using window.confirm over to Adapt.notify for displaying all SCORM errors - as the confirm dialog in some browsers (like Firefox) isn't great if the message to be displayed contains very long strings (such as the suspend data string).

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

No branches or pull requests

2 participants