Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Live video chat #6

Closed
tkosamja opened this issue Mar 24, 2018 · 6 comments
Closed

Live video chat #6

tkosamja opened this issue Mar 24, 2018 · 6 comments

Comments

@tkosamja
Copy link

Live video chat isn't styled.
I'll try to fix, I'm just letting this here to let you know and remind myself later on.

@mechalynx
Copy link
Collaborator

Mind mentioning what browser you're on, version and what userstyle extension you're using?

There are combinations that have known issues such as not being able to style iframes. I just tested on the popped out chat and it works, so if it's that, it's a known issue and, to my knowledge, a browser bug.

@tkosamja
Copy link
Author

tkosamja commented Mar 24, 2018

Hope you guys don't judge my music taste xD

<edit @RaitaroH>
@tkosamja omg it's terrible.

capture

Windows 10, Firefox 59, Stylus

@mechalynx
Copy link
Collaborator

Yeah this is known about Stylus on Firefox: openstyles/stylus#248 - I remember it coming up several times and it's a planned fix but I haven't been involved with Stylus development for the last few months so I don't know whether it's on track or not. It seems they're waiting for some API call to be implement on both browsers so the codebase isn't split. Rest assured this is not due to the userstyle or because Stylus is buggy - it's just a matter of how the API provided to the extension interacts with YouTube.

I think most of the devs on Stylus are using Chrome so stuff like this might get forgotten. You might want to open an issue on Stylus's repo (don't post in the one I linked, make a new one) but you're likely to get the answer I told you about: it's known and likely won't be fixed before the browser changes happen. This could be a regression as well so up to you. You can even join the discord and ask them directly for help just in case. Whatever you feel like, just be patient.

So until this is managed on Stylus's end, I'm afraid you'll just have to pop out chat or just deal with the unstyled chat. I'm closing this issue because it isn't the userstyle's fault but if anything else relevant comes up you can still comment.

@tophf
Copy link

tophf commented Apr 23, 2018

I thought this was fixed in Stylus 1.3.5 early March.
At least I can't repro the bug neither in FF stable nor nightly.

clip493

@mechalynx
Copy link
Collaborator

Hmm, not sure what was going on but even though on the popped up chat and on a separate tab with just the chat styles were applied correctly, if I had DeepDark Classic and DeepDark Material (the latter is intended to style the live chat) then the embedded chat doesn't get styled at all. I've got a 3rd personal style for the chat that wasn't applied either in this case.

However, if I disable DeepDark Classic, both of the other styles (Material and my personal one) are applied as normal. When I wrote the chat styling for Material I tested with both active, on the embedded chat, the popped out chat and on a separate tab and they all worked.

I'll modify the Classic style to exclude live chat since that works. If you want to try and reproduce just in case this is a Stylus bug, I'll link the commit where the issue is reproducible here after I make the change and test it.

@mechalynx mechalynx reopened this Apr 24, 2018
@mechalynx
Copy link
Collaborator

Was weirder, apparently it was only an issue on first view of the page - a refresh corrected it but it was always necessary to refresh once.

The exclusion fix resolves that so it's applied properly on the first view.

This is the repo at the previous commit, in case it's needed for testing: https://github.com/RaitaroH/YouTube-DeepDark/tree/bd32276550a43c56c911dafde10fd8833c37b3d1

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

No branches or pull requests

3 participants