1104927 - UITour: Add Loop conversation view target for email/copy link buttons

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Matthew N. [:MattN]]

We won't auto-open the conversation view since that is quite a bit of work so this depends on bug 1080943.

Things to handle:
* Update availableTargets to reflect whether a conversation window is open.
* Figure out what to do with multiple conversations. I propose we use the selected (non-minimized) one.
* Handle the conversation view popping out. I believe the tour always looks for targets in the same ChromeWindow at the moment so this may be tricky.

There is WIP code in attachment 8528515 [details] [diff] [review] for getting the selected conversation chatwindow.

Comment 1

[Matthew N. [:MattN]]

hideLoopPanelAnnotations will need to be update to not auto-hide annotations from the conversation view when the loop panel closes. We should close the targets in the converation window when it is closed though.

Comment 2

[Matthew N. [:MattN]]

Attachment: MozReview Request: bz://1104927/MattN

Comment 3

[Matthew N. [:MattN]]

/r/1411 - Bug 1104927 - UITour: Add Loop conversation view target for email/copy link buttons. r=Unfocused,mixedpuppy

Pull down this commit:

hg pull review -r e0d238caf1fdb43fcb92541a64c942498a6a9b78

Comment 4

[Matthew N. [:MattN]]

/r/1411 - Bug 1104927 - UITour: Add Loop conversation view target for email/copy link buttons. r=Unfocused,mixedpuppy

Pull down this commit:

hg pull review -r 6c1a6e6e0134c8b11f0a4011885dd2dc63d11caf

Comment 5

[Shane Caraveo (:mixedpuppy)]

Comment on attachment 8534857 [details]
MozReview Request: bz://1104927/MattN

Seems reasonable to me.

Comment 6

[Matthew N. [:MattN]]

agibson, in the interest of time/complexity I changed my mind and decided not to handle all of the ways that the annotation target could disappear with UITour and instead notify the webpage about the various events so it can call hideInfo(). Let me know if this won't work.

The event names for observe are listed at https://reviewboard.mozilla.org/r/1409/diff/#file18335 with the "Loop:" prefix.

Comment 7

[Blair McBride [:Unfocused] (UNAVAILABLE)]

https://reviewboard.mozilla.org/r/1409/#review853

::: browser/components/loop/MozLoopService.jsm
(Diff revision 2)
> -          UITour.notify("Loop:ChatWindowClosed");
> +          UITour.availableTargetsCache.clear();

I feel like we need a test for this, checking whether the conversation-related targets are in availableTargets or not.

Comment 8

[Matthew N. [:MattN]]

I really want to make a test for this but I think it will take so much time that it's not worthwhile since we will have manual testing of the tours when we use these targets. The problem is that I think I have to mock a bunch of loop code, possibly even fake servers in order to open a room.

Comment 9

[Blair McBride [:Unfocused] (UNAVAILABLE)]

Ouch :( Ok, fair enough.

Comment 10

[Blair McBride [:Unfocused] (UNAVAILABLE)]

https://reviewboard.mozilla.org/r/1409/#review855

Ship It!

Comment 11

[Matthew N. [:MattN]]

https://hg.mozilla.org/integration/fx-team/rev/6acf307a8620

Comment 12

[Matthew N. [:MattN]]

Bustage fix test update: https://hg.mozilla.org/integration/fx-team/rev/8778e0d6bba5

Comment 13

[Alex Gibson [:agibson]]

(In reply to Matthew N. [:MattN] from comment #6)
> agibson, in the interest of time/complexity I changed my mind and decided
> not to handle all of the ways that the annotation target could disappear
> with UITour and instead notify the webpage about the various events so it
> can call hideInfo(). Let me know if this won't work.
> 
> The event names for observe are listed at
> https://reviewboard.mozilla.org/r/1409/diff/#file18335 with the "Loop:"
> prefix.

Thanks, Matt

Will report back if I hit any issues

Comment 14

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/6acf307a8620
https://hg.mozilla.org/mozilla-central/rev/8778e0d6bba5

Comment 15

[Randell Jesup [:jesup] (needinfo me)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/71d6de6e9e16
https://hg.mozilla.org/releases/mozilla-aurora/rev/dfefc5d9930c

Comment 16

[Matthew N. [:MattN]]

https://hg.mozilla.org/releases/mozilla-beta/rev/52e7f251f16f

Comment 17

[Matthew N. [:MattN]]

https://hg.mozilla.org/releases/mozilla-beta/rev/b426dcaee75b

Comment 19

[Matthew N. [:MattN]]

Attachment: MozReview Request: Bug 1104927 - UITour: Add Loop conversation view target for email/copy link buttons. r=Unfocused,mixedpuppy