Disable pop-out feature in conversation window

RESOLVED FIXED

Status

defect
P1
normal
Rank:
19
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: RT, Assigned: mancas)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Given the e10s issue encountered on bug 1250534 we are considering disabling the pop-out feature on the conversation window to prevent the crash scenario from happening.
This bug will only be actioned if we find that bug 1250534 is not fixed on March 14th.

Acceptance criteria:
- Allow disablement of the pop-out button on the conversation through a preference
Blocks: 1250534
Rank: 19
Priority: -- → P1
Bug 1250534 isn't the issue here - that's almost fixed. Its bug 1245813 that's currently the big outstanding one.
Blocks: 1245813
No longer blocks: 1250534
Assignee: nobody → b.mcb
Status: NEW → ASSIGNED
Attachment #8730124 - Flags: review?(standard8)
Attachment #8730124 - Flags: review?(edilee)
Attachment #8730124 - Flags: review?(dcritchley)
Attachment #8730124 - Flags: review?(crafuse)
Attachment #8730124 - Flags: review?(standard8)
Attachment #8730124 - Flags: review?(edilee)
Attachment #8730124 - Flags: review?(dcritchley)
Attachment #8730124 - Flags: review?(crafuse)
Attachment #8730124 - Flags: review+
Note you can pop out the conversation window by dragging the title bar.  I believe this patch only disables the button, do we know if it also disables the drag behavior?
Flags: needinfo?(b.mcb)
Comment on attachment 8730124 [details] [review]
[loop] mancas:bug1255491 > mozilla:master

Hey Dave, as Ian has discovered when dragging the conversation window can be detached, so I've updated the patch to cover that case
Flags: needinfo?(b.mcb)
Attachment #8730124 - Flags: review+ → review?(dcritchley)
Comment on attachment 8730124 [details] [review]
[loop] mancas:bug1255491 > mozilla:master

Works as expected. Tested and wasn't able to drag and button didn't show. Ship it!
Attachment #8730124 - Flags: review?(dcritchley) → review+
Manu: Per discussion in standup today, please ship this with `loop.conversationPopOut.enabled` as `true` - there's other code landing soon that should fix the e10s issues this works around, but we want this just in case we need to turn things off.
Landed in master: https://github.com/mozilla/loop/commit/a83ace1bd8a0a17be979b316707b6e88d6d9b265
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Nice! Thanks, Manuel!
You need to log in before you can comment on or make changes to this bug.