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

RESOLVED FIXED in Thunderbird 14.0

Status

MailNews Core
Backend
--
major
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Karsten Düsterloh, Assigned: sgautherie)

Tracking

({regression})

Trunk
Thunderbird 14.0
regression
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
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.
(Assignee)

Updated

5 years ago
OS: Linux → All
Hardware: x86_64 → All
Version: unspecified → Trunk

Updated

5 years ago
Keywords: regression
(Assignee)

Updated

5 years ago
Blocks: 332151
tracking-seamonkey2.11: --- → ?
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

Comment 1

5 years ago
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)

Comment 2

5 years ago
Created attachment 616902 [details] [diff] [review]
(Av1) Move 'mail.advance_on_spacebar' preference from mail/ to mailnews/
[Checked in: Comment 5]
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+
(Assignee)

Comment 5

5 years ago
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]
(Assignee)

Updated

5 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
tracking-seamonkey2.11: ? → ---
Flags: in-testsuite-
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.