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] Do not clear url bar text on focus #10524

Closed
samartnik opened this issue Jun 30, 2020 · 5 comments · Fixed by brave/brave-core#5978
Closed

[Android] Do not clear url bar text on focus #10524

samartnik opened this issue Jun 30, 2020 · 5 comments · Fixed by brave/brave-core#5978

Comments

@samartnik
Copy link
Contributor

samartnik commented Jun 30, 2020

Description

As per discussion https://bravesoftware.slack.com/archives/C20HMJ62H/p1592687065037300
Do not clear url bar text on focus.

Steps to reproduce

  1. Go to any website
  2. Click on url bar
  3. Url itself is disappeared
    device-2020-06-30-133103

Actual result

Url itself is disappeared

Expected result

Url is shown and highlighted

Issue reproduces how often

Always

Issue happens on

  • Current Play Store version? yes
  • Beta build? yes

Device details

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

Brave version

1.10.x

Website problems only

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

Additional information

@samartnik samartnik added the OS/Android Fixes related to Android browser functionality label Jun 30, 2020
@samartnik samartnik self-assigned this Jun 30, 2020
@samartnik
Copy link
Contributor Author

samartnik commented Jun 30, 2020

Test plan:

  1. Go to any website
  2. Click on url bar
  3. Url should be shown and highlighted

@samartnik
Copy link
Contributor Author

After change
device-2020-06-30-133048

@samartnik
Copy link
Contributor Author

@jumde could you confirm if this behavior is ok or not brave/brave-core#5978 (comment)

@jumde
Copy link
Contributor

jumde commented Jul 8, 2020

@samartnik - comment here: brave/brave-core#5978 (comment)

@srirambv
Copy link
Contributor

srirambv commented Aug 6, 2020

Verification passed on OnePlus 6T with Android 10 running 1.12.106 x64 beta build


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


Verification passed on Asus Zenfone (x86) with Android 6 running 1.12.112 BraveModernx86.apk

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants