Closed
Bug 1130989
Opened 9 years ago
Closed 9 years ago
Collections details page is different in Mocks compered to MP-dev
Categories
(Marketplace Graveyard :: Consumer Pages, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
2015-02-10
People
(Reporter: vvalentina, Assigned: spasovski)
References
Details
Steps to reproduce: 1. Load details page for any collection from MP-dev homepage https://marketplace-dev.allizom.org/feed/collection/hippo?src=collection-element Expected results: There are no display or layout issues. Actual results: There is a white line after the last row of apps. Notes/Issues: Verified on MP-dev on FF38 (Win 7). Issue is also reproduce on FF OS 1.4 . Screenshot for this issue http://screencast.com/t/ml6QIILkb
Updated•9 years ago
|
Priority: -- → P3
Assignee | ||
Comment 1•9 years ago
|
||
https://github.com/mozilla/fireplace/pull/1012
Assignee: nobody → dspasovski
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 2•9 years ago
|
||
Verified as fixed on MP-dev FF38(Win 7, Android 4.2.1). Postfix screenshot http://screencast.com/t/vre4D1qZJ The issue was not fixed on FF OS 1.4 (Inari), is this intended? (the mp-dev app is up to date)
Reporter | ||
Updated•9 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 3•9 years ago
|
||
It was intentional to not affect mobile but I agree that it looks better with the last line missing there too. Fixed: https://github.com/mozilla/fireplace/commit/260456c
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 4•9 years ago
|
||
(In reply to Davor Spasovski [:spasovski] from comment #3) > It was intentional to not affect mobile but I agree that it looks better > with the last line missing there too. Fixed: > > https://github.com/mozilla/fireplace/commit/260456c Thanks! Verified as fixed on MP-dev on FF OS 1.4 (Inari) Closing bug.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•