mozilla / addons

☂ Umbrella repository for Mozilla Addons ✨
Other
125 stars 41 forks source link

Default text from search field is not removed while typing #10390

Closed ValentinaPC closed 7 years ago

ValentinaPC commented 7 years ago

Steps to reproduce:

  1. Load AMO-dev homepage on your Android device https://addons-dev.allizom.org/en-US/android/
  2. Type something on the search field

Expected results: Default text from search field is erased.

Actual results: Search term is displayed over the default text from the search field.

Notes/Issues:

Verified on AMo-dev FF53(Android 6.0.1) Video fro this issue: videotogif_2017 05 22_12 26 14

muffinresearch commented 7 years ago

@ValentinaPC how easy is this to repro. I can't reproduce it on a nexus 7. Can you repro in production?

muffinresearch commented 7 years ago

Do you see any js console errors when this behaviour occurs?

ValentinaPC commented 7 years ago

@muffinresearch : I wanted to record another thing, while bumped over it again - this is how hard is to reproduce it. :) Unfortunately I did not had the device connected to my computer to see the console.

ValentinaPC commented 7 years ago

I've tried to reproduce this, but not being able to reproduce the entire thing: only the part when the cursor is put in front of the magnifier and the default text is not removed, but while typing everything goes as expected. I think has something to do with changing the languages. Nothing in the console, for now. :( Video: videotogif_2017 05 25_16 37 56

ValentinaPC commented 7 years ago

@muffinresearch : Reproduced it again today using AMO-dev FF53(Android 7.1.2) From logs:

05-29 10:58:09.721 2017-2188/? E/BrcmNfcJni: nfaConnectionCallback: unknown event ????
05-29 10:58:14.929 504-504/? E/SurfaceFlinger: releasePendingBuffer failed: Unknown error -1 (1)

Here are all logs, before and after this issue was reproducing - https://pastebin.mozilla.org/9022963 Hope it helps!

kumar303 commented 7 years ago

Thanks for the logs. I didn't see anything obvious in there but here they are again since the pastebin link will expire: 9022963.txt

ValentinaPC commented 7 years ago

I think this is related to mozilla/addons#10425 and its root cause, because this is happening again in -dev.

ValentinaPC commented 7 years ago

Happening in stage, today. Video: videotogif_2017 07 13_10 36 13

apoorvasingh17 commented 7 years ago

Hi, can I work on this issue?

tofumatt commented 7 years ago

@apoorvasingh17 You're welcome to work on it though this one seems quite hard to reproduce and will require an Android phone to replicate. Are you able to reproduce the issue locally?

apoorvasingh17 commented 7 years ago

@tofumatt I guess I will have to pass on this one as I have just started contributing and would like to start off with something easier.

tofumatt commented 7 years ago

That’s okay, good thinking. I didn’t want you to get overwhelmed on your first issue. Right now we don’t seem to have a lot of issues filed as good first bugs but I’ll see if I can find some that are good to start with!

muffinresearch commented 7 years ago

@ValentinaPC is this still an issue now that the header has be reimplemented?

ValentinaPC commented 7 years ago

@muffinresearch : did not see it in the new header. We can close this as fixed.