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

[Android] YT Improvements: Default video quality settings #32842

Open
timchilds opened this issue Sep 7, 2023 · 6 comments · May be fixed by brave/brave-core#24164
Open

[Android] YT Improvements: Default video quality settings #32842

timchilds opened this issue Sep 7, 2023 · 6 comments · May be fixed by brave/brave-core#24164
Assignees
Labels
enhancement OS/Android Fixes related to Android browser functionality QA/Yes release-notes/include

Comments

@timchilds
Copy link

timchilds commented Sep 7, 2023

Description

See Default video quality settings (cellular vs. wifi) and Requirements sections here:
https://bravesoftware.slack.com/archives/CELA35G5S/p1694106384308469?thread_ts=1671572756.471319&cid=CELA35G5S

06/13 Update

Updated requirements based on slack conversation:

  • Default to Allow over Wi-Fi
  • When On or Allow over Wi-Fi = choose 720p resolution (or the highest available below 720p).
  • When Off = yt will decide resolution or user can change to max via yt player settings (which could be 4k).

New setting copy:

Toggle
Title: High Definition Playback
Subtitle: Enables HD-quality resolution when available

Screen
Header: HD Quality Settings
Note: Video quality above HD can be enabled via the youtube.com player
^^ show this note below the on/wifi/off radio buttons

@timchilds timchilds added enhancement QA/Yes release-notes/include OS/Android Fixes related to Android browser functionality labels Sep 7, 2023
@tapanmodh tapanmodh linked a pull request Jun 12, 2024 that will close this issue
24 tasks
@timchilds
Copy link
Author

timchilds commented Jun 13, 2024

Updated requirements based on slack conversation:

  • Default to Allow over Wi-Fi cc @tapanmodh in case you missed this update on slack.
  • When On or Allow over Wi-Fi = choose 720p resolution (or the highest available below 720p).
  • When Off = yt will decide resolution or user can change to max via yt player settings (which could be 4k).

New setting copy:

1st Screen

  • Title: High Definition Playback
  • Subtitle: Enables HD quality resolution when available.

2nd Screen

  • Header: HD Quality Settings
  • Note: Video quality above HD can be enabled via the youtube.com player. show this note below the on/wifi/off radio buttons

image

@rmcfadden3 please review the copy when you have time.

cc @anthonypkeane @deeppandya @tapanmodh

@rmcfadden3
Copy link

Looks good overall. Few minor things:

  • I believe we need capitalization for Up Next and End Card, as those are branded features within YouTube. So like Blocks the Up Next and End Card elements on youtube.com
  • Need title case on the Block Distracting Elements section header (since this is iOS)
  • Need serial comma on the Block Distracting Elements body copy. So like Blocks the sign in, menu, comments, and video info elements on youtube.com
  • The body for Enable Background Audio could use some commas. Probably like this: Allows audio to play when you switch to another tab in Brave or another app, or when your device screen is off
  • Since we're clearly talking about Brave, should the Widevine one end with …media content in Brave rather than …media content in the browser ?

Also, is there a reason for the inconsistent punctuation? Some of these descriptions have periods, while others do not. I'm okay either way, as long as we're consistent. Perhaps, since 2 of these end with youtube.com we should just go with no periods throughout.

@timchilds
Copy link
Author

timchilds commented Jun 13, 2024

@rmcfadden3 we can address your concerns about the already live features in another GH issue.

I was referring to this copy:

1st Screen
Title: High Definition Playback
Subtitle: Enables HD quality resolution when available.

2nd Screen
Header: HD Quality Settings
Note: Video quality above HD can be enabled via the youtube.com player. show this note below the on/wifi/off radio buttons.

@rmcfadden3
Copy link

@rmcfadden3 we can address your concerns about the already live features in another GH issue.

I was referring to this copy:

1st Screen
Title: High Definition Playback
Subtitle: Enables HD quality resolution when available.
2nd Screen
Header: HD Quality Settings
Note: Video quality above HD can be enabled via the youtube.com player. show this note below the on/wifi/off radio buttons.

Seems alright. Just need a hyphen in first subtitle (HD-quality rather than HD quality). And then, note my comment above about punctuation: need to be consistent, so can probably remove the periods from both of your new subtitles here.

@timchilds
Copy link
Author

timchilds commented Jun 13, 2024

Thanks @rmcfadden3.

@deeppandya @tapanmodh final copy:

Title: High Definition Playback
Subtitle: Enables HD-quality resolution when available

Header: HD Quality Settings
Note: Video quality above HD can be enabled via the youtube.com player
^^ show this note below the on/wifi/off radio buttons

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement OS/Android Fixes related to Android browser functionality QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants