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)

Avenir
x86_64
Windows 7
defect

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
Priority: -- → P3
https://github.com/mozilla/fireplace/pull/1012
Assignee: nobody → dspasovski
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
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)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
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 ago9 years ago
Resolution: --- → FIXED
(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.