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] Tapping 'X' on Site settings pages doesn't do anything #12181

Closed
Brave-Matt opened this issue Oct 16, 2020 · 2 comments · Fixed by brave/brave-core#8372
Closed

[Android] Tapping 'X' on Site settings pages doesn't do anything #12181

Brave-Matt opened this issue Oct 16, 2020 · 2 comments · Fixed by brave/brave-core#8372
Assignees
Labels
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/Yes regression release-notes/include

Comments

@Brave-Matt
Copy link

Description

On Android, visiting any Site Setting page (ex. Menu --> Settings --> Site settings --> Javascript) and tapping the x on the top right of the page does not close out the menu as intended. Currently, the button is not functional and seems to do nothing. This happens for all pages inside Site Settings.

Steps to reproduce

  1. Launch Brave
  2. Visit Menu --> Settings --> Site settings --> Javascript or some other Site Setting page
  3. Tap x on the top right

Actual result

Nothing happens.

Expected result

Menu should close out.

Issue reproduces how often

Easily/every time.

Version/Channel Information:

  • Can you reproduce this issue with the current Play Store version?
    Yes
  • Can you reproduce this issue with the current Play Store Beta version?
    Yes
  • Can you reproduce this issue with the current Play Store Nightly version?
    Yes

Device details

(Several)

  • Install type (ARM, x86):
  • Device type (Phone, Tablet, Phablet): Phone
  • Android version: Android 10

Brave version

v1.15.73

Website problems only

  • Does the issue resolve itself when disabling Brave Shields? N/A
  • Does the issue resolve itself when disabling Brave Rewards? N/A
  • Is the issue reproducible on the latest version of Chrome? N/A

Additional information

xnoworky

@Brave-Matt Brave-Matt added the OS/Android Fixes related to Android browser functionality label Oct 16, 2020
@srirambv srirambv added this to Backlog in Android General via automation Oct 19, 2020
@anthonypkeane anthonypkeane added the priority/P2 A bad problem. We might uplift this to the next planned release. label Mar 10, 2021
@anthonypkeane anthonypkeane moved this from Backlog to P2 in Android General Mar 10, 2021
@alexsafe
Copy link

Solved by brave/brave-core#8372

Android General automation moved this from P2 to Done/Closed Mar 29, 2021
@bbondy bbondy added closed/duplicate Issue has already been reported and removed closed/duplicate Issue has already been reported labels Apr 6, 2021
@bbondy bbondy added this to the 1.24.x - Nightly milestone Apr 6, 2021
@stephendonner
Copy link

stephendonner commented Apr 28, 2021

Verified FIXED using Brave - Beta 1.24.76, Chromium 90.0.4430.85 on Android 9.

Steps:

  1. Launched Brave
  2. Tapped on the 3-dots menu, --> Settings --> Site settings --> JavaScript
  3. Tapped X on the top right

Confirmed the Site Settings page closed, and I was returned to the new-tab page.

example example example
Screenshot_20210428-145712 Screenshot_20210428-145718 Screenshot_20210428-145722

Verification passed on Samsung Tab A with Android 10 running 1.24.81 x64 build

  • Verified close button on Site settings page closes menu
Android.Tab.Close.settings.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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/Yes regression release-notes/include
Projects
Android General
  
Done/Closed
Development

Successfully merging a pull request may close this issue.

7 participants