Earlier quoted context omitted.
Looks like it's all fixed now :)
Yeah, I added some real XSS protection. Before today, I thought it would be cool to enable some html :)
Yes, indeed the party is over. Going for beers now ;)
Show HN: MapChat – A simple location based chat in 300 lines of code
61–70 of 122 posts
Re: Show HN: MapChat – A simple location based chat in 300 lines of code
#62Great work! Obviously ~300 lines of code isn't enough to do a full-fledged slick interface...but one suggestion that presumably would be easy to implement: a button that closes all current word-balloons.
I was also thinking a "reply" button for threaded discussions between users...but then that would kill the fun simplicity of the app...I just like seeing where HN users are currently this morning. And also, the ways people try to obfuscate XSS attacks
Re: Show HN: MapChat – A simple location based chat in 300 lines of code
#63Would be neat to have private rooms.
Re: Show HN: MapChat – A simple location based chat in 300 lines of code
#64Great work! Obviously ~300 lines of code isn't enough to do a full-fledged slick interface...but one suggestion that presumably would be easy to implement: a button that closes all current word-balloons. I was also thinking a "reply" button for threaded discussions between users...but then that would kill the fun simplicity of the app...I just like seeing where HN users are currently this morning. And also, the ways…
Thanks! The button to close all current balloons is at the top of my list :)
I'm afraid that a reply button might take out some of the fun and the simplicity, but I will conciser that.
Re: Show HN: MapChat – A simple location based chat in 300 lines of code
#65Re: Show HN: MapChat – A simple location based chat in 300 lines of code
#66Would be neat to have private rooms.
There are private rooms, but the UI do not support that yet.
Just add #word to the url and get your private chat map.
i.e. http://idoco.github.io/map-chat#test
Re: Show HN: MapChat – A simple location based chat in 300 lines of code
#67Earlier quoted context omitted.
This is what I did for now msg.text = msg.text.replace('>','') .replace(' I will think of a more clever solution next week :)
That's super easy to get around. It only replaces the first occurrence of each.
You are right that was very silly of me. I got some real XSS filter instead.
Re: Show HN: MapChat – A simple location based chat in 300 lines of code
#68I think something of a timeout on the message bubbles is needed.. As to not cloud the screen too much. Nice work though :)
Was just about to mention that.
Re: Show HN: MapChat – A simple location based chat in 300 lines of code
#69I think something of a timeout on the message bubbles is needed.. As to not cloud the screen too much. Nice work though :)
Yes, this is at the top of my list. Thanks :)