search

mathew-kurian / Scribe.js

:scroll: Node.js logging made simple! Online access to logs and more...
https://mathew-kurian.github.io/Scribe.js/
MIT License
284 stars 45 forks source link

Live Reload for Web Interface #44

Open jimbuck opened 9 years ago

jimbuck commented 9 years ago

This might have been a design consideration, but I'd really like a live reload on the web portal. Something as simple as a configurable interval to reload would do the trick, just something that would allow me to leave the logs up and not have to constantly refresh to view them. It really is a great interface, especially when fullscreen.

guillaumewuip commented 9 years ago

Hi @JimmyBoh. You're right, it something that should be done. It's really not complicated (just a js setInterval or the angular equivalent). As I'm really busy these days and maybe @bluejamesbond is too, feel free to send a pull request :smiley:

jimbuck commented 9 years ago

I actually am going to work on this tonight. Any quirks when setting up the dev environment?

mathew-kurian commented 9 years ago

Live reload with angular might require some perf optimizations since you don't want to reload each row on update. Instead, you want to insert the missing rows at the bottom or top depending on how you sort it. I would also recommend that the added rows have a highting animation such as this.

jimbuck commented 9 years ago

Agreed. I've implement an service before that uses interval polling to get the latest log entries from a C# WebAPI. Is infinite scrolling a thing, or are all logs loaded at once?

mathew-kurian commented 9 years ago

@JimmyBoh I think you should use a socket api such as socket.io in order to send data back and forth. no, there is no infinite scrolling concept at this point. However, we would be open to it.

jimbuck commented 9 years ago

@bluejamesbond Awesome, sounds good to me. I know some people get worried when adding extra libraries.

jimbuck commented 9 years ago

I'm guessing this is a merge issue in the .gitignore? If so is there a grunt file that I need for development?

mathew-kurian commented 9 years ago

@JimmyBoh That seems to be a merge issue. I personally don't use a gruntfile. I can't speak for @guillaumewuip tho

guillaumewuip commented 9 years ago

Probably an error coming from my personnal gruntfile, for jshint, guys.

jimbuck commented 9 years ago

@guillaumewuip No problem, I was able to get everything up and running, and started to integrate Socket.IO into the server. I will be quite busy over the next two weeks but will try to squeeze in some development when I can!

mathew-kurian commented 9 years ago

Sounds like a plan!