Jim Mathies [:jimm] Reporter
	Description • 11 years ago

I'm not sure what I did to get this, but this seems kinda wrong. See attached screenshot.

	Jim Mathies [:jimm] Reporter
	Comment 2 • 11 years ago

STR:

using a device with out a hardwired keyboard

1) tap the nav bar from firefox start
2) enter a couple characters using the soft keyboard
3) tap empty space in firefox start to hide the keyboard

result: validation rect on the nav bar edit.

	Jim Mathies [:jimm] Reporter
	Comment 3 • 11 years ago

P1 since this is easy to reproduce and shouldn't be hard to fix.

	Rodrigo Silveira [:rsilveira] [:rodms]
	Comment 4 • 11 years ago

I think JWilde's fix for bug 811406 handled this, right?

	Frank Yan (:fryn)
	Comment 5 • 11 years ago

(In reply to Rodrigo Silveira [:rsilveira] from comment #4)
> I think JWilde's fix for bug 811406 handled this, right?

Yup.

	Frank Yan (:fryn)
	Comment 6 • 11 years ago

JWilde, I know you fixed this in bug 811406, but the selector that you used is kinda slow: :invalid falls into the slowest "universal" bucket, and you're using a descendant selector. Could you do a one-line adjustment to the selector just to make it a bit faster? Add a class name to the element that matches :invalid if needed.

Thanks! :)

	Frank Yan (:fryn)
	Comment 7 • 11 years ago

My bad for not catching that in the code review, btw. :P

	Jonathan Wilde [:jwilde] Assignee
	Comment 8 • 11 years ago

:D

	Jonathan Wilde [:jwilde] Assignee
	Comment 9 • 11 years ago

Thought that previous patch worked when it didn't actually.

	Frank Yan (:fryn)
	Comment 10 • 11 years ago

Comment on attachment 763079 [details] [diff] [review]
patch v1.1

:D

	Jonathan Wilde [:jwilde] Assignee
	Comment 11 • 11 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/2e6a5ba61bdf

	Marco Mucci [:MarcoM]
	Comment 12 • 11 years ago

Hi Jonathan, can you provide a point estimate for this Defect Story.

	Jonathan Wilde [:jwilde] Assignee
	Comment 13 • 11 years ago

p=1

	Ryan VanderMeulen [:RyanVM]
	Comment 14 • 11 years ago

https://hg.mozilla.org/mozilla-central/rev/2e6a5ba61bdf

	Kamil Jozwiak [:kjozwiak]
	Comment 15 • 11 years ago

Went through the following "Defect" for iteration #9 testing without any issues. Used the following build:

http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-07-08-03-11-14-mozilla-central/

- Went through the test case that has been added in comment 2 without any issues
- Typed in a few letters in the "Navigation App Bar" and selected the white space under the awesome screen and then the "Navigation App Bar" several times going back and forth (both touch/mouse)
- Went back and forth between the "Navigation App Bar" and the white spaces after typing a few letters in the URL bar while on a website (both touch/mouse)
- Ensured that pasting something into the "Navigation App Bar" and then going between the "Navigation App Bar" & the white spaces worked without any issues
- Ensured the above cases worked in "Filled" view without any issues
- Ensured the above cases worked in "Snapped" view without any issues

	Kamil Jozwiak [:kjozwiak]
	Comment 16 • 11 years ago

Went through the following "Defect" for iteration #10 testing without any issues. Used the following build:

http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-07-19-03-02-04-mozilla-central/

- Went through the test case that has been added in comment 2 without any issues
- Went through the test cases that have been added in comment 15 without any issues

	Samvedana (:Samvedana)
	Comment 17 • 11 years ago

User Agent: Mozilla/5.0 (Windows NT 6.2; Win64; x64; rv:26.0) Gecko/20100101 Firefox/26.0
Build ID: 20130807030216
Built from http://hg.mozilla.org/mozilla-central/rev/1fb5d14e8348

WFM
Tested on windows 8 using latest nightly for iteration-11. Followed steps provided in comment2 and got expected result.

	Samvedana (:Samvedana)
	Comment 18 • 11 years ago

User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0
Build ID: 20130816030205
Built from http://hg.mozilla.org/mozilla-central/rev/1ed5a88cd4d0

WFM
Tested on windows 8 using latest nightly for iteration-12. Followed steps provided in comment2 and got expected result.