Closed
Bug 757825
Opened 12 years ago
Closed 12 years ago
Chat UI not correctly hidden on existing profiles
Categories
(Thunderbird :: Instant Messaging, defect)
Tracking
(thunderbird13+ fixed, thunderbird14+ fixed)
RESOLVED
FIXED
Thunderbird 15.0
People
(Reporter: florian, Assigned: florian)
Details
Attachments
(1 file)
1.07 KB,
patch
|
standard8
:
review+
standard8
:
approval-comm-aurora+
standard8
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
I had a report on IRC that on Tb13b2 with an existing profile the "Tools -> Chat Status" and "Go -> Chat" menu items weren't hidden. There was this error in the error console: Error: document.getElementById(aId) is null Line 683 File: chat-messenger-overlay.js http://mxr.mozilla.org/comm-beta/source/mail/components/im/content/chat-messenger-overlay.js#683 The same problem didn't happen with a fresh profile. I suspect for some reason the Chat button wasn't present in the toolbar. A trivial fix seems to just add a null check before attempting to hide an element.
Assignee | ||
Updated•12 years ago
|
tracking-thunderbird13:
--- → ?
tracking-thunderbird14:
--- → ?
Comment 1•12 years ago
|
||
I'm the one that hit this. If I can do anything to assist debugging, just let me know -- I can even package up my profile if that's helpful.
Updated•12 years ago
|
Assignee | ||
Comment 2•12 years ago
|
||
Trivial fix (just adding a null check). [Approval Request Comment] This bug is currently quite visible on beta where the Chat feature is pref'ed off, so we need this fix on aurora and beta. Steps to reproduce on current trunk: - with Chat enabled (the default on trunk), customize the main toolbar and remove the "Chat" button. - disable chat (set mail.chat.enabled to false) - restart - notice the "document.getElementById(aId) is null" error in the error console, and the "Go -> Chat" and "Tools -> Chat status" aren't hidden.
Assignee: nobody → florian
Attachment #626774 -
Flags: review?(mbanner)
Attachment #626774 -
Flags: approval-comm-beta?
Attachment #626774 -
Flags: approval-comm-aurora?
Updated•12 years ago
|
Attachment #626774 -
Flags: review?(mbanner)
Attachment #626774 -
Flags: review+
Attachment #626774 -
Flags: approval-comm-beta?
Attachment #626774 -
Flags: approval-comm-beta+
Attachment #626774 -
Flags: approval-comm-aurora?
Attachment #626774 -
Flags: approval-comm-aurora+
Assignee | ||
Comment 3•12 years ago
|
||
https://hg.mozilla.org/comm-central/rev/9c8ea067b3f2
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 15.0
Assignee | ||
Comment 4•12 years ago
|
||
https://hg.mozilla.org/releases/comm-aurora/rev/9f435453c7bb https://hg.mozilla.org/releases/comm-beta/rev/40ca16a22853
status-thunderbird13:
--- → fixed
status-thunderbird14:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•