Nitzan Tomer

Results 21 comments of Nitzan Tomer

@chiptus Yeah, you're right, there's a missing `be` in there. Feel free to open a PR for fixing it, but I doubt that it will help you get to know...

@chiptus I've been told that the info about how to run the local environment doesn't work, and I still haven't had the time to get into it. BUT you can...

Some logging services: - https://www.loggly.com/ - https://clevertap.com/ - https://mixpanel.com/ - https://logrocket.com/ - https://sentry.io/welcome/ - https://www.logdna.com/ - https://papertrailapp.com/ - https://logentries.com/ - https://stackify.com/ - https://www.sumologic.com/ - https://www.graylog.org/ (taken from a thread here:...

Another discussion on this: https://www.facebook.com/groups/jsisrael/permalink/1600454106664928/ Two more additions: - https://trackjs.com/ - https://www.bugsnag.com/

@creativeChips @amir-arad WDYT?

I'm not sure whether or not those errors need to be shown in the console. As @creativeChips said, it can be useful to have them when debugging but there's currently...

Can you change back the whitespace to tabs? Currently there are too many changes because of the re-formatting.

This is pretty cool. Sorry that it took time to get to it. Seems good to me, other than the comment about the `isEmpty` utility function. WDYT?

You're right, it breaks the scrollbar. I flowed your advice and changed it to: `Math.ceil(this.element.scrollTop + this.element.offsetHeight) >= this.element.scrollHeight` I also updated the scrollbar lib version and removed a use...

@TomZelazny Sorry, I forgot about it... Re-opened it, if you still wanna take a look. I agree that a `css` only solution is the best, but it might be tricky..