Comment on attachment 9004091 When changing layout, display the message asynchronously.
Review of attachment 9004091: -----------------------------------------------------------------
::: mail/base/content/msgMail3PaneWindow.js @@ +240,5 @@ > messenger.setWindow(null, null); > messenger.setWindow(window, msgWindow); > + setTimeout(function DelayReloadMessage() { > + ReloadMessage(); > + }, 0);
nit: no need to name the function. Also, 0 is the default so not needed.
Anyway, it may or may not fix the issue depending on your system. So a bit of a hack papering over the real problem. If we do take it, add a comment to explain why it's there.
Comment on attachment 9004091
When changing layout, display the message asynchronously.
Review of attachment 9004091: ------- ------- ------- ------- ------- ------- ------- ------- --
-------
::: mail/base/ content/ msgMail3PaneWin dow.js setWindow( null, null); setWindow( window, msgWindow); age() {
@@ +240,5 @@
> messenger.
> messenger.
> + setTimeout(function DelayReloadMess
> + ReloadMessage();
> + }, 0);
nit: no need to name the function. Also, 0 is the default so not needed.
Anyway, it may or may not fix the issue depending on your system. So a bit of a hack papering over the real problem.
If we do take it, add a comment to explain why it's there.