Peertube: Comment posted negative seconds ago

[scanlime]

I assume there may be a small time desynchronization between server and client, but in any case i don't think this is what we want to be displaying:

I'd expect we could solve this by rounding any times more recent than a couple seconds to "just now" or something, ignoring negative values effectively.

[Poslovitch]

I may work on this one this weekend

[Poslovitch]

It seems it happens when publishing videos as well...

[Chocobozzz]

@Poslovitch any news on this? :)

[Poslovitch]

@Chocobozz hi, sorry, I'll have to dismiss my contribution. I don't feel confident enough with this language. Sorry again :(

[zacharystenger]

Hi, I'd like to fix this issue. I believe I've got a proper fix here. It uses Math.max to make a negative seconds result equal to zero instead. It also gets rid of a redundant Math.floor operation.

[rigelk]

@zacharystenger thanks ! Please make a pull request.

[Chocobozzz]

Fixed by @zacharystenger in https://github.com/Chocobozzz/PeerTube/commit/ec24796ea83dbe15863c4f074844abbb4cdd5e08