More of a case to abstract this out, then.


On Thu, Dec 5, 2013 at 11:54 AM, Adam Hyland <notifications@github.com>wrote:

> In apps/sms/js/thread_ui.js:
>
> > @@ -752,7 +760,11 @@ var ThreadUI = global.ThreadUI = {
> >            },
> >            {
> >              l10nId: 'discard-message',
> > -            method: discard
> > +            method: function ondiscard() {
> > +              Drafts.delete(MessageManager.draft);
> > +              MessageManager.draft = null;
>
> I think we also need to call something like
>
>           if (Threads.active) {
>             ThreadListUI.updateThread(Threads.active);
>           }
>
> Here. So when we delete a draft we re-render the thread (regardless of our
> cache in ThreadListUI). I have to look at other cases (e.g. where we aren't
> in an active thread) but that seems to resolve the problem of deleted
> drafts still being previewed in the thread list.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/rwaldron/gaia/pull/2/files#r8135422>
> .
>



-- 
Evelyn Eastmond
evhan55@alum.mit.edu