

I don’t think I said that OP was wrong (in fact I’m pretty sure I agreed with OP that it was annoying as a fully abled user until I figured out what was going on), but to elaborate on my earlier point just slightly - once a horizontal slider element is in focus it will always use left and right arrow keys to move if they’re pressed. Up and down always works to change the volume because there are no other elements when the video player is selected that are waiting for an input.
If they made the volume slider vertical instead of horizontal it would end up working actually better in this case (because the vertical slider would instead wait for up and down arrows), although that probably wouldn’t look as “nice”, but I’m not a UX dev so it could probably be jazzed up to look nice.
It’s okay. I’ve got a few side projects in node and I also don’t see anything too out of the ordinary here. If anything this looks more like someone’s first project with some “make it work” rather than being perfect and pretty.
I don’t understand the hostility. Even the bits that this guy talks about (duplicated code?) nothing looked duplicated - sure the names of the folders are the same but that’s more of a naming convention problem when you have a client that needs to connect to servers (or in this case has screens and functionality to connect to servers).
And now I’ve probably spent more time and brainpower on this than I should have just because I was confused.