282877 - spacebar no longer works in mailnews three column layout

Status: RESOLVED DUPLICATE of bug 282938

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Sander]

In mailnews, the spacebar servers the double function of going page-down in a
message, or moving to the next unread message when at the end of the current
message. This functionality stopped working somewhere between 2005020106 and
2005021605. (will narrow down further in a bit)

Javascript console says:
  Error: contentWindow has no properties
  Source File: chrome://messenger/content/mailWindowOverlay.js
  Line: 1750

Comment 1

[Tuukka Tolvanen (sp3000)]

wfm seamonkey trunk 2005-02-18-18Z on linux, tested in news

Comment 2

[Sander]

Tested more; I couldn't reproduce with a new profile, until I changed to the
three column layout. Tuukka confirms.

Comment 3

[Tuukka Tolvanen (sp3000)]

Actually, I find it broken in both non-default layouts.

Comment 4

[Sander]

This regressed between 2005020106 and 2005020204. My suspicious checkin radar
doesn't seem to be working for that regression window though. Anyone who can
spot a likely cause?

Comment 5

[Sander]

If it isn't too late yet, I think this should maybe block 1.8b1 as well. Then
again, it's only a beta, so probably not.

Comment 6

[[e-mail]]

This looks like a regression from my patch for bug 104532 :-(
I'll see if I can find out what I regressed, in the meantime as a workaround:
msgMail3PaneWindow.js has a function UpdateMailPaneConfig
At the start of the if block, add
getMessageBrowser().setAttribute("type", "content");
And at the end add
getMessageBrowser().setAttribute("type", "content-primary");

Comment 7

[[e-mail]]

My patch in bug 282938 seems to fix this.

Comment 8

[Sander]

This was indeed fixed by the patch in bug 282938

*** This bug has been marked as a duplicate of 282938 ***