Closed
Bug 1011616
Opened 10 years ago
Closed 10 years ago
Join Chat dialog box won't close with Auto-join this Chat Room checked
Categories
(Thunderbird :: Instant Messaging, defect)
Tracking
(thunderbird31 fixed)
RESOLVED
FIXED
Thunderbird 32.0
Tracking | Status | |
---|---|---|
thunderbird31 | --- | fixed |
People
(Reporter: walts48, Assigned: clokep)
References
Details
(Whiteboard: [regression])
Attachments
(3 files, 1 obsolete file)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
aleth
:
review+
standard8
:
approval-comm-aurora+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
Details | Diff | Splinter Review |
Sign-on to a chat account
Click Join Chat
Enter a Channel name in the Join Chat dialog box
Place a check mark to enable 'Auto-join this Chat Room'
Click OK
Chat room is joined, but the Join Chat dialog box doesn't close
Reporter | ||
Comment 1•10 years ago
|
||
2nd attempt at a visual
Assignee | ||
Updated•10 years ago
|
Attachment #8424015 -
Attachment is obsolete: true
Assignee | ||
Comment 2•10 years ago
|
||
Are there any errors in the error console? Does this only happen when you check "Auto-join this Chat Room"?
Reporter | ||
Comment 3•10 years ago
|
||
Only happens when I check "Auto-Join this Chat Room", otherwise it closes.
No errors in the Error Console.
Thanks for removing my mistake.
Assignee | ||
Comment 4•10 years ago
|
||
I'm able to reproduce this on Daily.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Whiteboard: [regression]
Assignee | ||
Comment 5•10 years ago
|
||
Looks like this needs to port https://hg.mozilla.org/comm-central/rev/ab630c829a4b I'm going to sync im & mail versions of this file.
Assignee: nobody → clokep
Status: NEW → ASSIGNED
Assignee | ||
Comment 6•10 years ago
|
||
Porting the focusing of the conversation seemed like too big of a change for this bug, but the current diff is pretty small.
Attachment #8425941 -
Flags: review?(aleth)
Assignee | ||
Comment 7•10 years ago
|
||
Assignee | ||
Comment 8•10 years ago
|
||
By the way this is a regression due to bug 920801, commit https://hg.mozilla.org/comm-central/rev/01a788d85004
Blocks: 920801
Updated•10 years ago
|
Attachment #8425941 -
Flags: review?(aleth) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 9•10 years ago
|
||
Comment on attachment 8425941 [details] [diff] [review]
Patch v1
https://hg.mozilla.org/comm-central/rev/678a0897be2a
[Approval Request Comment]
Regression caused by (bug #): bug 920801 (https://hg.mozilla.org/comm-central/rev/01a788d85004)
User impact if declined: Unable to add chat rooms to the auto-join list from the "Join Chat" menu. Current aurora will be the next version of TB to go to the release channel.
Testing completed (on c-c, etc.): Committed to c-c, I've tested this by hand, we can let it bake on c-c and ask the reporter to ensure the bug is fixed though.
Risk to taking this patch (and alternatives if risky): I see this as pretty low risk, it's porting a change that has been in Instantbird since Oct. 14, 2013 (~7 months).
Attachment #8425941 -
Flags: approval-comm-beta?
Attachment #8425941 -
Flags: approval-comm-aurora?
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Target Milestone: --- → Thunderbird 32.0
Updated•10 years ago
|
Attachment #8425941 -
Flags: approval-comm-aurora? → approval-comm-aurora+
Comment 10•10 years ago
|
||
status-thunderbird31:
--- → fixed
Comment 11•10 years ago
|
||
Comment on attachment 8425941 [details] [diff] [review]
Patch v1
This landed before the merge, so no need for a separate beta landing.
Attachment #8425941 -
Flags: approval-comm-beta?
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•