-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Install Widevine prompt should include more DRM background information #9907
Comments
cc: @rmcfadden3 If you think we should revisit update our widevine screen. |
@rebron — sorry for the delayed reply on this one. I'm actually OK with the text here. Yes, it's long. But 1) it seems good that users have all the info here rather than having to click through to a help page, and 2) my sense is this is not a commonly seen message, so it wouldn't affect very many users. But correct me if I'm mistaken in those assumptions and you think we should edit. |
Pretty sure Netflix, Amazon Prime and the like are commonly used and everyone would see this text on their first try. |
Ah, didn't realize how frequently this message might appear. Will investigate new / better copy, and a corresponding help doc with more info that people could click through to. |
cc: @Brave-Matt We'll be updating the text here and will need to Support documentation to match. |
@rebron — here's new copy for the Widevine message. Note the "Learn more" will point to a support page that explains more. Also note the "X" beside the header is meant to allow user to close the message without making a decision. Header:
Body:
CTAs:
|
@rmcfadden3 What did you want to do with the Change later at brave://settings/extensions CTA? Do we want to just include it as text and link in the body or how did you want to incorporate that one. |
@rmcfadden3 @rebron How about this? |
@simonhong — that looks really good. The only thing I'd suggest is changing the order in the message footer, so it's clear that "Learn more" will tell you about Widevine, rather than settings in general. So the footer could read: "[Learn more about Widevine](URL goes here). Change later at brave://settings/extensions." |
BTW, I just checked in w/ Mattches to see about publishing a support doc so you have an endpoint URL for that "Learn more about Widevine" text. |
fix brave/brave-browser#9907 Permission bubble for widevine has custom footnote style.
fix brave/brave-browser#9907 Permission bubble for widevine has custom footnote style.
Verified
Steps:
Verified
Verified
|
Description
Running EME on a site for the first time makes Brave ask the user, whether they'd like to install it. While that's a good thing, it also has a big piece of text, explaining:
However, it lacks information on:
Steps to Reproduce
Actual result:
Expected result:
Expected a short popup with one-sentence description and a link to read more. Something more like Firefox, but with an actual explanation.
Example text:
The "read more" should link to a Brave article explaining what a DRM is, how it works, why it's bad, how Widevine works and how to install/enable/disable/remove it.
Reproduces how often:
Easily reproduced
Brave version (brave://version info)
1.10.60
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
Related: #2791
The text was updated successfully, but these errors were encountered: