Skip to content

Conversation

@Kansattica
Copy link
Contributor

I would like to use Syncplay to watch a movie with a friend for a podcast where we watch together, pause to talk about things, and assemble it into a slide slow later. Syncplay printing when the media was paused in its log would help with this immensely.

@Et0h
Copy link
Contributor

Et0h commented Apr 6, 2022

This seems like a good idea to me - a small tweak that makes the notification more useful. I tested the code and it works for me.

I'd be interested in people's thoughts on whether the proposed format of "Bob paused (00:05)" is better than going with "Bob paused at 00:05". The latter is more in line with the format for "Bob jumped from 14:04 to 12:56" format used elsewhere and avoids conflict with "(MM:SS)" being the format used to display the duration when announcing "Bob is playing 'Example.mkv' (23:40) in room: 'Videos'".

It could still use () for other languages for now as that doesn't require a translation, but we could add something to the stub that they should consider tweaking it to follow the English format.

@Kansattica
Copy link
Contributor Author

I originally wrote "Bob paused at 00:05" and changed it to the other phrasing because I didn't have the expertise to translate it into the other languages. If it's preferable to have a complete sentence for English and leave the others as stubs for now, I'm happy to make the change.

@Et0h Et0h merged commit 2ef8570 into Syncplay:master Apr 7, 2022
@Et0h
Copy link
Contributor

Et0h commented Apr 7, 2022

It's a simple tweak so I'm happy to make the change myself now that I've confirmed you are happy with the wording.

Et0h added a commit that referenced this pull request Apr 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants