Bug in example/lib/defaultHtmlStepUi.js

[harshkhandeparkar]

Flash Message During Step Removal Typo Bug

image-sequencer/examples/lib/defaultHtmlStepUi.js

Line 19 in bf13d58

$('#stepRemovedNotification').DOMNotification.fadeIn(500).delay(200).fadeOut(500);

DOMNotification is not defined and is a typo

[welcome]

Thanks for opening your first issue here! Please follow the issue template to help us help you 👍🎉😄
If you have screenshots to share demonstrating the issue, that's really helpful! 📸 You can make a gif too!

[gitmate]

GitMate.io thinks the contributor most likely able to help you is @jywarren.

Possibly related issues are #408 (Typo in info.json), #401 (Typo in CONTRIBUTING.MD), and #25 (Issues With ImageManipulation.js).

[gitmate]

GitMate.io thinks the contributor most likely able to help you is @jywarren.

Possibly related issues are #408 (Typo in info.json), #401 (Typo in CONTRIBUTING.MD), and #25 (Issues With ImageManipulation.js).

[grvsachdeva]

hey @harshkhandeparkar, could you please give your Email ID?

[harshkhandeparkar]

Why do you need by Email ID?

[grvsachdeva]

Actually, we can talk here only:

@harshkhandeparkar thanks so much for your enthusiasm in the weekly check-in! Sorry we made some last minute changes, but we thought it over a bit and felt that a) we worried that some memes might end up making people feel bad, even by accident, and b) we have a strong need for reviewers anyways. I hope this makes sense why we deleted your comment from weekly check-in and didn't want this to be discouraging - we are super grateful for your help in collaboratively hosting the community. Thanks so much for your time and keep up the great work!

[harshkhandeparkar]

Oh Np. Even I thought I would delete my comment but you had already done that. I was hesitant to comment at first but I saw your comment and I thought it won't be much of an awkward thing. But anyways, thank you, Np, welcome etc. 😊

[grvsachdeva]

Great!