[imp] IMP (Dynamic) - Date in preview window does not show the time

[Michael M Slusarz]

Quoting Luis Felipe Marzagao/Andamentos <lfbm.andamentos at gmail.com>:

> Is it intended for imp to not display the time of the message in
> preview window when the msg isn't from the same day?
>
> I can't see why it would hurt to show the message's time in preview
> window, since there is a lot of space in there

It is wasted information.  It has nothing to do with amount of space  
available.  It isn't good UI to simply "fill up" the space will  
available data.  (It is the same reason we no longer display e-mail  
addresses for recipients that have name information).  Especially when  
the date is also available in the message list.

I would agree that time of the message may be important for messages  
within the last few days or so.  But for messages sent weeks, months,  
or years ago, it is completely useless information.

> It's a useful information and this way the user wouldn't have to click
> two times on the message just to see the time it has arrived.
>
> I got a lot of complaining from users and solved the matter by
> changing line 206 of imp/lib/Views/ShowMessage.php from:
>
> 206:     $result['localdate'] =  
> htmlspecialchars($imp_mailbox_ui->getDate($envelope->date));
>
> to:
>
> 206:     $result['localdate'] =  
> htmlspecialchars($imp_mailbox_ui->getDate($envelope->date,  
> IMP_Ui_Mailbox::DATE_FULL));
>
> But It's a hack, maybe an ungly one, I don't know.

As mentioned above, this can not be a general solution.  I wouldn't  
mind if maybe the time information was provided in a  
hover/title/tooltip display.

michael

___________________________________
Michael Slusarz [slusarz at horde.org]