Closed
Bug 1176700
Opened 9 years ago
Closed 9 years ago
Unnecessary gray padding at the bottom of a website detail page
Categories
(Marketplace Graveyard :: Consumer Pages, defect, P4)
Tracking
(Not tracked)
VERIFIED
FIXED
2015-06-23
People
(Reporter: madalin.cotetiu, Assigned: trishul.goel)
References
Details
Attachments
(1 file)
6.76 MB,
video/mp4
|
Details |
Step to reproduce: 1. Open detail page for a website. 2. Scroll to the bottom of the page. Expected results: The page can only be scrolled as long as there are information to be displayed. Actual results: The page can be scrolled more than necessary and a gray padding can be seen at the bottom of the page. Notes/Issues: Issue can be reproduced in marketplace-dev Verified in FFOS 2.0, 2.1, 3.0 (Flame), FF41 In Android 4.2.1
Updated•9 years ago
|
Priority: -- → P4
Comment 1•9 years ago
|
||
https://github.com/mozilla/fireplace/pull/1332
Assignee: nobody → kngo
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 2•9 years ago
|
||
The issue is still reproducing in FFOS 2.0, 3.0 (Flame), FF41(Android 4.2.1) Please view the attached video. Reopening bug.
Assignee: kngo → madalin.cotetiu
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 3•9 years ago
|
||
I don't think we can fix this easily, the mobile nav depends on the page having a min-height. I forget the exact reason why, but wontfixing this.
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → WONTFIX
Comment 4•9 years ago
|
||
Actually, it should be fixed. But it's not part of MEOW.
Reporter | ||
Comment 5•9 years ago
|
||
The issue is also reproducing in any category landing page where there are only 1 or 2 websites: http://screencast.com/t/Wug8I2dvF
Updated•9 years ago
|
Severity: normal → minor
Can the following be solution? https://github.com/mozilla/fireplace/pull/1386 Basically removing the scroll for less content pages.
Kindly review the PR, if this is the required solution
Flags: needinfo?(kngo)
Merged #1386
Assignee: madalin.cotetiu → trishul.goel
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Flags: needinfo?(kngo)
Resolution: --- → FIXED
Reporter | ||
Comment 10•9 years ago
|
||
Verified as fixed in FFOS 2.0 (Flame) FF42(Android 4.2.1) Closing bug.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•