[Tickets #11091] Re: Re-add show-images-by-address-book

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/11091
------------------------------------------------------------------------------
  Ticket             | 11091
  Updated By         | Michael Slusarz <slusarz at horde.org>
  Summary            | Re-add show-images-by-address-book
  Queue              | IMP
  Version            | Git develop
  Type               | Enhancement
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


Michael Slusarz <slusarz at horde.org> (2012-03-24 08:45) wrote:

> Oh, and while I'm there, though this not exactly the same issue, it  
> would be great to automatically load the images of the current  
> message after whitelisting a sender.

I thought it did?  Although... it probably doesn't since  
imageunblock.js does the bare minimum and delegates other features to  
the base page script via the unblock event.

> This used to work when adding sender to the address book too.

This definitely isn't true for dynamic view.  The addContact AJAX call  
does not define a callback.

This doesn't seem like expected behavior to a user either.  And it's  
really not that big of a deal warranting a potentially complicated  
mechanism to do this for a single message - I can't imaging a user  
complaining that they had to manually display images in this situation.