Closed Bug 1185083 Opened 9 years ago Closed 9 years ago

Empty panel, Remote Tabs displays errant cell dividing line

Categories

(Firefox for iOS :: Home screen, defect)

Other
iOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
fxios + ---

People

(Reporter: tecgirl, Assigned: bmunar)

References

Details

Attachments

(2 files)

In landscape, a cell divider appears at the bottom of the Remote Tabs panel (below the content). It should not appear.

See attached.
Blocks: 1183213
tracking-fxios: --- → ?
Attached file PR
Attachment #8635606 - Flags: review?(bnicholson)
Assignee: nobody → bmunar
Comment on attachment 8635606 [details] [review]
PR

Not a huge fan of this fix...the "3 * cell.bounds.width" value being used feels completely arbitrary. Also, I noticed that I still see a small bit of the separator in the bottom right corner when I test on the iPad Air profile.

I'm not familiar these iOS UI hacks -- sleroux might have some ideas.
Attachment #8635606 - Flags: review?(bnicholson) → review?(sleroux)
Comment on attachment 8635606 [details] [review]
PR

There's actually a neat trick you can do to get rid of this line instead of setting a large inset. UITableViews by default will always show the separators unless there is a footer specified. If there is a footer, the bottom line won't appear. To fix this, I usually just set a footer for the first section and set it to a basic UIView.
Comment on attachment 8635606 [details] [review]
PR

Looks good!
Attachment #8635606 - Flags: review?(sleroux) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: