Closed Bug 746745 Opened 8 years ago Closed 8 years ago

[SeaMonkey] Space for moving to next unread doesn't work (JavaScript error: chrome://messenger/content/mailWindowOverlay.js, line 2311)

Categories

(MailNews Core :: Backend, defect, major)

defect
Not set
major

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 14.0

People

(Reporter: mnyromyr, Assigned: sgautherie)

References

Details

(Keywords: regression)

Attachments

(1 file)

JavaScript error: chrome://messenger/content/mailWindowOverlay.js, line 2311: NS_ERROR_UNEXPECTED: Component returned failure code: 0x8000ffff (NS_ERROR_UNEXPECTED) [nsIPrefBranch.getBoolPref]

when hitting space to move to next unread message - and no message switching happens. Obviously, the pref mail.advance_on_spacebar lacks a default value in SeaMonkey.

This is caused by the Thunderbird bug 332151.
OS: Linux → All
Hardware: x86_64 → All
Version: unspecified → Trunk
Keywords: regression
Blocks: 332151
Component: MailNews: Message Display → Backend
No longer depends on: 332151
Product: SeaMonkey → MailNews Core
QA Contact: message-display → backend
Summary: Space for moving to next unread doesn't work (JavaScript error: chrome://messenger/content/mailWindowOverlay.js, line 2311) → [SeaMonkey] Space for moving to next unread doesn't work (JavaScript error: chrome://messenger/content/mailWindowOverlay.js, line 2311)
Target Milestone: --- → Thunderbird 14.0
sorry about that.  you're all undoubtedly more familiar with the code base than i am so i'll leave the patch to you.
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #616902 - Flags: review?(bwinton)
Comment on attachment 616902 [details] [diff] [review]
(Av1) Move 'mail.advance_on_spacebar' preference from mail/ to mailnews/
[Checked in: Comment 5]

I _think_ this is okay, but Standard8 is really the person who would know, so I'm redirecting the review request his way.

Thanks,
Blake.
Attachment #616902 - Flags: review?(bwinton) → review?(mbanner)
Comment on attachment 616902 [details] [diff] [review]
(Av1) Move 'mail.advance_on_spacebar' preference from mail/ to mailnews/
[Checked in: Comment 5]

Looks fine.
Attachment #616902 - Flags: review?(mbanner) → review+
Comment on attachment 616902 [details] [diff] [review]
(Av1) Move 'mail.advance_on_spacebar' preference from mail/ to mailnews/
[Checked in: Comment 5]

http://hg.mozilla.org/comm-central/rev/b8a4823c1ca3
Attachment #616902 - Attachment description: (Av1) Move 'mail.advance_on_spacebar' preference from mail/ to mailnews/ → (Av1) Move 'mail.advance_on_spacebar' preference from mail/ to mailnews/ [Checked in: Comment 5]
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.