Closed Bug 1100378 Opened 10 years ago Closed 9 years ago

Standalone room view should have an option to retry connecting to a room if something went wrong

Categories

(Hello (Loop) :: Client, defect, P1)

defect
Points:
2

Tracking

(firefox35+ fixed, firefox36 fixed, firefox37 fixed)

RESOLVED FIXED
mozilla37
Iteration:
37.1
Tracking Status
firefox35 + fixed
firefox36 --- fixed
firefox37 --- fixed
backlog Fx35+

People

(Reporter: standard8, Assigned: rgauthier)

References

Details

(Whiteboard: [standalone][rooms])

Attachments

(2 files, 1 obsolete file)

      No description provided.
We should introduce an option to the standalone room to try and reconnect to the room if something goes wrong. This is probably a simple retry button, along with an appropriate re-connect.
Summary: Something went wrong in a room → Standalone room view should have an option to retry connecting to a room if something went wrong
Blocks: 1074659
backlog: --- → Fx35+
Priority: -- → P2
Whiteboard: [standalone][rooms]
What I think we need to do here is:

- In StandaloneRoomInfoArea#_renderContent add a new "button" in the ROOM_STATES.FAILED case - similar to the way it is done in ROOM_STATES.INIT/READY (there's an existing button implementation).

- When the button is pressed, automatically re-join the room, i.e. call this.props.joinRoom - which dispatches the roomJoin action.

I'm a little worried about edge cases, but I think we should be fine, as when we switch to ROOM_STATES.FAILED, we already do a lot of cleanup.

However, it'd be worth some manual testing just to check for anything unexpected.
Assignee: nobody → rgauthier
Attachment #8530293 - Flags: ui-review?(nperriault)
Added a test
Attachment #8530293 - Attachment is obsolete: true
Attachment #8530293 - Flags: ui-review?(nperriault)
Attachment #8530310 - Flags: review?(nperriault)
Comment on attachment 8530310 [details] [diff] [review]
Retry button for Hello standalone

Review of attachment 8530310 [details] [diff] [review]:
-----------------------------------------------------------------

The code is good but needs a little style tweak; r=me with proper styling of the Retry button.

::: browser/components/loop/standalone/content/js/standaloneRoomViews.jsx
@@ +127,5 @@
>              <div className="room-inner-info-area">
>                <p className="failed-room-message">
>                  {this._getFailureString()}
>                </p>
> +              <button className="btn btn-join btn-retry"

This creates a big pink button; I don't think that's what we want in terms of styling :)
https://dl.dropboxusercontent.com/spa/0jzcrapltxzlqds/ef6xhy23.png

I think you can reuse the btn-info style as used line 76 of that file.
Attachment #8530310 - Flags: review?(nperriault) → review+
Addressed comment
Attachment #8530322 - Flags: review?(nperriault)
[Tracking Requested - why for this release]:
This patch is for the standalone app. We may choose to uplift it for the sake of shared code with desktop.  Marking this as tracking Fx35 so we get this fixed before Fx35 goes to Release.
Comment on attachment 8530322 [details] [diff] [review]
Retry button for Hello standalone

Review of attachment 8530322 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good.
Attachment #8530322 - Flags: review?(nperriault) → review+
Hi Niko,  t0keshu's out this week.  are you comfortable landing this for this week?
Flags: needinfo?(nperriault)
Priority: P2 → P1
(In reply to sescalante from comment #9)
> Hi Niko,  t0keshu's out this week.  are you comfortable landing this for
> this week?

Sure, I'm just waiting for the tree to reopen ;) https://treestatus.mozilla.org/fx-team
Flags: needinfo?(nperriault)
https://hg.mozilla.org/integration/fx-team/rev/fc97b1d13a0a
Iteration: --- → 37.1
Points: --- → 2
Target Milestone: --- → mozilla37
https://hg.mozilla.org/mozilla-central/rev/fc97b1d13a0a
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment on attachment 8530322 [details] [diff] [review]
Retry button for Hello standalone

Approval Request Comment
[Feature/regressing bug #]: N/A

[User impact if declined]: Possible merge conflicts if a patch touches standalone and shared code.

[Describe test coverage new/current, TBPL]: N/A

[Risks and why]: No risk - standalone only - no impact on build

[String/UUID change made/needed]: none
Attachment #8530322 - Flags: approval-mozilla-beta?
Attachment #8530322 - Flags: approval-mozilla-aurora?
Attachment #8530322 - Flags: approval-mozilla-beta?
Attachment #8530322 - Flags: approval-mozilla-beta+
Attachment #8530322 - Flags: approval-mozilla-aurora?
Attachment #8530322 - Flags: approval-mozilla-aurora+
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.