Closed
Bug 1035445
Opened 10 years ago
Closed 10 years ago
add detail feed view to transonic previews
Categories
(Marketplace Graveyard :: Curation Tools, defect, P5)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: krupa.mozbugs, Assigned: spasovski)
References
()
Details
Attachments
(1 file)
1.27 MB,
image/png
|
Details |
steps to reproduce: 1. Load https://marketplace-dev.allizom.org/curate/manage/shelves/felwjkb 2. Notice the feed preview 3. Add a few apps and see if the preview updates observed behavior: Feed preview for operator shelves doesn't show the listed apps. See screenshot.
Updated•10 years ago
|
Assignee: nobody → kngo
Priority: -- → P2
Comment 1•10 years ago
|
||
Operator shelves never show their constituent apps on the feed, and the preview currently only shows the feed representation of a feed item, not the detail. That said, we probably should be showing both. Davor, thoughts on that?
Flags: needinfo?(dspasovski)
Comment 2•10 years ago
|
||
It's been a TODO.
Updated•10 years ago
|
Assignee: kngo → nobody
Flags: needinfo?(dspasovski)
Summary: Feed preview for operator shelves is missing apps → add detail feed view to transonic previews
Comment 3•10 years ago
|
||
I don't think I have the "bandwidth" for this, and Chuck is on PTO.
Assignee: nobody → dspasovski
Comment 4•10 years ago
|
||
For implementation, I would move the feed detail view templates in Fireplace to feed_item.html. Pass the feed macros a "landing=True" flag. If that flag is set, have it include another macro that contains detail view specific stuff.
Updated•10 years ago
|
Priority: P2 → P5
Assignee | ||
Comment 5•10 years ago
|
||
https://github.com/mozilla/transonic/commit/02e5239b0d5d8adeb2335f54f7b1f3468028f6d0
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•