Closed
Bug 1147132
Opened 11 years ago
Closed 11 years ago
Keyboard covers top sites content
Categories
(Firefox for iOS :: Home screen, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: bnicholson, Assigned: bnicholson)
References
Details
Attachments
(1 file)
Hit this issue on a fresh profile with <6 total entries, with the phone in landscape mode. The keyboard covers the tiles, and trying to touch/scroll doesn't make it go away. Looks like there are a couple of problems:
1) Our collection view height calculation is wrong if there are <6 entries, which breaks scrolling.
2) Setting "keyboardDismissMode = .OnDrag" makes the keyboard go away if we scroll the view, but the view isn't scrollable if the content doesn't stretch beyond what's visible.
|
Assignee | |
Updated•11 years ago
|
Summary: Keyboard covers to sites content → Keyboard covers top sites content
|
|
Assignee | |
Comment 1•11 years ago
|
||
Attachment #8582651 -
Flags: review?(sarentz)
|
||
Comment 2•11 years ago
|
||
Comment on attachment 8582651 [details] [review]
Pull request
LGTM
Attachment #8582651 -
Flags: review?(sarentz) → review+
|
|
Assignee | |
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•