Closed Bug 1177597 Opened 9 years ago Closed 9 years ago

Search screen does not show on iOS9

Categories

(Firefox for iOS :: General, defect)

Product:
Firefox for iOS
Component:
General
Platform:
Other
iOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
fxios + ---

People

(Reporter: st3fan, Assigned: sahil.profile)

References

Details

(Whiteboard: ios9)

Attachments

(1 file)

Pull Request
47 bytes, text/x-github-pull-request
bnicholson
: review+
Details | Review
STR: 1) Navigate to some site 2) Tap the location bar 3) Start typing Expected: The search screen is shown. You see search results for your default search engine. Actual: Nothing happens. You stay on the home screen with the tiles. Only seems to happen on iOS9. No hints in the console.
In iOS9 due to some bug `UIKeyInput` method `insertText` do not call. I have created pull request which resolve the issue. https://github.com/mozilla/firefox-ios/pull/715
Attached file Pull Request —
Attachment #8632556 - Flags: review?(bnicholson)
Comment on attachment 8632556 [details] [review] Pull Request Thanks for the pull request! On the right track, but we need to figure out a fix for this that doesn't flash the input each time a character is typed.
Attachment #8632556 - Flags: review?(bnicholson) → feedback+
Assignee: nobody → sahil.profile
Status: NEW → ASSIGNED
Comment on attachment 8632556 [details] [review] Pull Request Sorry, looks like the flicker issue is an existing bug on master. I'll give this another look.
Attachment #8632556 - Flags: feedback+ → review?(bnicholson)
Comment on attachment 8632556 [details] [review] Pull Request Seems to work well on English keyboards, though breaks Japanese as mentioned in the PR.
Attachment #8632556 - Flags: review?(bnicholson) → feedback+
@brian thanks for the comments. Have not seen your comments yesterday in bugzilla. Resolved the flicker issue. I will am also looking in to japanese character issue. Will commit the changes after testing.
@brian update the pull request. Fixed flicker issue and japanese character issue.
Comment on attachment 8632556 [details] [review] Pull Request Going to try to look at this today when I get a chance.
Attachment #8632556 - Flags: feedback+ → review?(bnicholson)
This patch is a bit more complicated than I was expecting :) I probably won't have time to do a review today, so I'll get to it early next week. Sorry for the delay!
@brian. Updated the pull request to resolve commits.
Whiteboard: ios9
@brian. Updated the pull request to resolve commits.
Hi @brian. Updated the pull request to resolve commits.
Comment on attachment 8632556 [details] [review] Pull Request iOS 9 fix looks OK, though I'll keep thinking about an alternative fix to the multistage input issue. I wonder if we can considerably simplify the flicker fix, though, as mentioned in the PR. Thoughts? Thanks for taking the time to investigate all of this!
Attachment #8632556 - Flags: review?(bnicholson) → feedback+
Comment on attachment 8632556 [details] [review] Pull Request Hi Brian Thanks for suggestion. I have updated the pull request. Simplified the patch. Thanks for your help
Attachment #8632556 - Flags: review?(bnicholson)
@brian Updated the pull request to resolve conflicts.
Comment on attachment 8632556 [details] [review] Pull Request This looks and feels great! Thanks for all the hard work, codester! :)
Attachment #8632556 - Flags: review?(bnicholson)
Attachment #8632556 - Flags: review+
Attachment #8632556 - Flags: feedback+
https://github.com/mozilla/firefox-ios/commit/f496e89a2ed85d34bb93cfd51d12b7551814f4a9
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Depends on: 1189127
Depends on: 1189413
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: