Closed
Bug 858654
Opened 11 years ago
Closed 11 years ago
Add Thumbnail Input to Project Editor
Categories
(Webmaker Graveyard :: Popcorn Maker, defect)
Webmaker Graveyard
Popcorn Maker
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: cade, Assigned: cade)
References
Details
Attachments
(1 file)
No description provided.
Assignee | ||
Comment 1•11 years ago
|
||
Missed the description - We need a tool to drag/drop or specify a link to an image that will be used as a thumbnail or potentially a poster frame for a project.
Status: NEW → ASSIGNED
Comment 2•11 years ago
|
||
maybe im missing this but why not use the same functionality from the image editor?
Comment 3•11 years ago
|
||
That's exactly what it was. The code exists for this all to just drop in place. Truthfully Chris should be able to just open a PR with it pretty quick considering the fact the main UI changes to the editor have landed.
Assignee | ||
Comment 4•11 years ago
|
||
https://github.com/mozilla/butter/pull/1658 still needs work, but I would like some feedback.
Attachment #735150 -
Flags: feedback?(jon)
Attachment #735150 -
Flags: feedback?(david.humphrey)
Comment 5•11 years ago
|
||
Comment on attachment 735150 [details] [review] Front-end changes for thumbnail input. Front-end is fine, but the backend hasn't been changed out yet, so lets just wait on this.
Attachment #735150 -
Flags: feedback?(jon)
Attachment #735150 -
Flags: feedback?(david.humphrey)
Attachment #735150 -
Flags: feedback+
Comment 6•11 years ago
|
||
Does this feature a default thumbnail? I'd like it to - the popcorn maker logo is fine for now.
Assignee | ||
Comment 7•11 years ago
|
||
:brett - It doesn't have one in it's current state, I'll modify the changes so that every new project gets the popcorn logo as a default thumbnail (for now)
Assignee | ||
Comment 8•11 years ago
|
||
Opened a new pull, github got confused after I rebased. https://github.com/mozilla/butter/pull/1672
Comment 9•11 years ago
|
||
Chris, I just reviewed over the code again. I want to move along and get this and bug 858739 landed. Can you land this? I can take over for the backend changes, but I'm guessing you still have the original code for them sitting somewhere.
Assignee | ||
Comment 10•11 years ago
|
||
Staged on Master: https://github.com/mozilla/butter/commit/9b0d773c325553e8a54690fda402afd954615757
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Attachment mime type: text/plain → text/x-github-pull-request
You need to log in
before you can comment on or make changes to this bug.
Description
•