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

Share button in Brave Shields does nothing #29856

Closed
SergeyZhukovsky opened this issue Apr 19, 2023 · 5 comments · Fixed by brave/brave-core#18485
Closed

Share button in Brave Shields does nothing #29856

SergeyZhukovsky opened this issue Apr 19, 2023 · 5 comments · Fixed by brave/brave-core#18485
Assignees
Labels
Android 8 - Test/Verification Checking on an older Android device to make sure everything is working as expected Android 13 Impacting Android 13 devices bug OS/Android Fixes related to Android browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA Pass - Android Tab QA Pass - Android x86 QA/Test-All-Platforms QA/Yes release-notes/include

Comments

@SergeyZhukovsky
Copy link
Member

SergeyZhukovsky commented Apr 19, 2023

It happens on a stable 1.50.x channel as well. It looks like it regressed for some time.

Edit: It looks it works on some devices. I tried on Pixel 6 Android 13 and it doesn't work.

@SergeyZhukovsky SergeyZhukovsky added bug priority/P2 A bad problem. We might uplift this to the next planned release. QA/Yes release-notes/include OS/Android Fixes related to Android browser functionality labels Apr 19, 2023
@deeppandya
Copy link

it's known issue for android 13. could be related to runtime permissions

@kjozwiak
Copy link
Member

@deeppandya @SergeyZhukovsky I couldn't reproduce the issue I saw re: the icon being greyed out but I can confirm that I can reproduce the same issue on Pixel 6 running Android 14. I can reproduce it without/with Notifications enabled.

Brave | 1.50.121 Chromium: 112.0.5615.138 (Official Build) (32-bit)
--- | ---
Revision | 2258f2fcaeb07bc95c992b7ca47f0b43ead9865c-refs/branch-heads/5615@{#1282}
OS | Android 13; Build/UPB1.230309.014

Even though the above mentions Android 13, it's Android 14 as they haven't updated the OS version yet. Appears as UpsideDownCake.

STR/Cases:

  • launch 1.50.121 Chromium: 112.0.5615.138 and run through onboarding
  • open any website and tap on the brave shields panel, tap on the Share icon and notice nothing happens

Reproduceable with All Brave notifications enabled and disabled. Seems to be working fine via Android 12. Tapping on the Share button initiates a request for permissions. I think @timchilds was right re: the above regressing with Android 13 permissions introduction. Example of Android 12 working:

Screen_Recording_20230420_003903_Brave.mp4

@kjozwiak kjozwiak added the Android 13 Impacting Android 13 devices label Apr 20, 2023
@brave-builds brave-builds added this to the 1.53.x - Nightly milestone May 16, 2023
@hffvld hffvld added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Jun 5, 2023
@hffvld
Copy link
Contributor

hffvld commented Jun 5, 2023

Verified on Pixel 2 XL, Pixel 6, Pixel 7, Galaxy A32 5G and Galaxy Tab S8 using version(s):

Device/OS: 
- Pixel 2 XL [taimen-user 8.1.0 OPM2.171026.006.H1 release-keys]
- Pixel 6 [oriole-user 13 TQ2A.230505.002 release-keys]
- Pixel 7 [panther_beta-user 14 UPB2.230407.014 release-keys]
- Galaxy A32 5G [a32xsq-user 13 TP1A.220624.014 release-keys]
- Galaxy Tab S8 [gts8wifixx-user 13 TP1A.220624.014 release-keys]
Brave build: 1.53.81
Chromium: 114.0.5735.90 (Official Build) beta (64-bit)
Revision: 386bc09e8f4f2e025eddae123f36f6263096ae49-refs/branch-heads/5735@{#1052}

STEPS:

  1. Install Brave > Launch Brave
  2. Run through onboarding
  3. Open any website > Tap on the Brave Shields panel
  4. Tap on the Share icon > Verify

ACTUAL RESULTS:

  • Verified that Brave Shields Share button is working fine on Android 8, Android 13 and Android 14

Android 8

timestamp_14-33-07_14-34-06.mp4

Android 13

timestamp_14-37-11_14-37-53.mp4

Android 14

timestamp_14-41-13_14-42-03.mp4

@kjozwiak
Copy link
Member

kjozwiak commented Jul 6, 2023

The above requires 1.52.130 or higher for 1.52.x verification 👍 Removing the QA Pass labels as the above will need to be rechecked with 1.52.x.

@kjozwiak kjozwiak added Android 8 - Test/Verification Checking on an older Android device to make sure everything is working as expected and removed QA Pass - Android 8+ labels Jul 6, 2023
@Uni-verse
Copy link
Contributor

Uni-verse commented Jul 6, 2023

Verified on Samsung Galaxy S21 5G, Samsung Galaxy Tab S7, Pixel 5 using the following build(s):

Brave	1.52.130 Chromium: 114.0.5735.198 (Official Build) (32-bit) 
Revision	c3029382d11c5f499e4fc317353a43d411a5ce1c-refs/branch-heads/5735@{#1394}
OS	Android 13; Build/TP1A.220624.014; 33; REL
Brave	1.52.130 Chromium: 114.0.5735.198 (Official Build) (32-bit) 
Revision	c3029382d11c5f499e4fc317353a43d411a5ce1c-refs/branch-heads/5735@{#1394}
OS	Android 8.0.0; Build/OSR1.180418.026; 26; REL
  • Verified tapping share button in Brave Shields opens up share window
GS21 Tab S7 Pixel 5 (Android 8)
screenshot-1688661035527 screenshot-1688662862794 Screenshot 2023-07-06 at 1 29 52 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android 8 - Test/Verification Checking on an older Android device to make sure everything is working as expected Android 13 Impacting Android 13 devices bug OS/Android Fixes related to Android browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA Pass - Android Tab QA Pass - Android x86 QA/Test-All-Platforms QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants