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)
(deleted),
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
|
||
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
•