Syncplay / syncplay

Client/server to synchronize media playback on mpv/VLC/MPC-HC/MPC-BE on many computers
http://syncplay.pl/
Apache License 2.0
2.1k stars 214 forks source link

Add a chatbox #54

Closed jurplel closed 8 years ago

jurplel commented 9 years ago

That sounds like a pretty convenient feature to me. Perhaps combine it with the notifications section on the left.

Et0h commented 9 years ago

Thanks for your suggestion, jeep70.

Having chat within Syncplay is certainly something we have considered in the past. One of the reasons it has not already been implemented is because chat facilities have the potential to be abused, and we didn't want to spend all our time working on moderation for it when established chat solutions already existed.

Last time I thought about it I identified two potential ways around that issue:

  1. Have Syncplay-based chat enabled on a server-by-server basis, so if you are running a server with friends who won't abuse the feature then you can enable it, but we wouldn't have it enabled on the public server and so don't have to worry about moderation; and/or
  2. Build an IRC client into Syncplay, so then it can all take place on someone else's server which already has established forms of oversight. May add complexity, but would allow for people to chat on our public server without us having to worry about moderation.

It'd be useful to hear what people's thoughts are on chat functionality, and on how to manage its potential for abuse. Do you like either option? Both? Do you have other ideas?

iansampaio commented 9 years ago

I like the idea of having a chat. Perhaps it'd be less intrusive if it was integrated to the player, and shown along with the video timeline like Soundcloud does:

screenshot 2015-03-13 12 55 11

Et0h commented 9 years ago

That's an interesting idea, but Syncplay can't really integrate stuff into the media player like that. Adding such functionality into every player Syncplay supports would be a nightmare, let alone maintaining such functionality if it has to be a branch because the people who made the player didn't want it complicating their code. If it were to be added to the Syncplay window it'd be a lot work for something which people mostly wouldn't see.

That said, if Syncplay allowed logging of what comments were made during a Syncplay session, what file was being played at the time, and the file position at the time, then someone could create a utility to generate such a representation of the video after-the-fact.

Logging with context would also be useful for people wanting to blog their views about shows they are watching (especially if chats could be logged to a file). It would be fun if there was a way to automatically have screencaps of when people made the comment, maybe including the comment itself, but again that might be better off as a separate utility that parses the chat log then the video rather than something within Syncplay. Arguably that's fairly specialist so not a top priority, but it would be pretty neat.

If there were chat it makes sense that it shows up in the main video via OSD and is also recorded in the Syncplay notification window. It is possible that people could toggle and untoggle what can be seen in the window so people can only see chat or hide chat.

Et0h commented 8 years ago

See https://github.com/Syncplay/syncplay/pull/90 and chat branch which has an alpha of this feature.