[imp] bugs in mail previews in mailbox.php

Marko Djukic mdjukic at horde.org
Thu Mar 4 05:27:19 PST 2004


michael,

not sure what the logic was, but the preview is not working properly, at least
for the inline (non-tooltip) version.

line 710 in imp/mailbox.php, why does this line strip out ' ' and replace them
with  ?
   $msg['preview'] = str_replace(' ', ' ', $msg['preview']);

the end result is a mailbox screen which runs off 18 screens scrolls to the
right.

or maybe it was a typo in line 708? is it supposed to read:
   if ($preview_tooltip) {
(note lack of '!' test)
this would fix the error that crops up saying that $strip_preview is not
defined, since in line 397 it is only defined if $preview_tooltip is true but
then it is used in 709 when $preview_tooltip is false...

i tried following chora's annotations but couldn't figure out what was the
thinking behind the code. so if you could help out, thanks.



--
Marko Djukic
Horde Project (http://horde.org)


More information about the imp mailing list