This only affects Rooms, not calls. So it doesn't affect fx34 calling

Its currently centred to the page rather than the remote video. IMO, centring it to the remote video would also make it feel awkward. I think there's two options here: - Centre it on the remote video - See if we can display the local video with a grey background, to give a bit more balance. We probably want UX input here, so NI to Sevaan.

[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.

(In reply to Mark Banner (:standard8) from comment #1) > Its currently centred to the page rather than the remote video. IMO, > centring it to the remote video would also make it feel awkward. > > I think there's two options here: > > - Centre it on the remote video > - See if we can display the local video with a grey background, to give a > bit more balance. > > We probably want UX input here, so NI to Sevaan. It should be centers on the remote view, like where the "Join the conversation" button is here: http://cl.ly/image/1o17222g1g2i

need owner

[Tracking Requested - why for this release]: After triage we've moved this to blocking Fx36 (though we'd love to fix it this week).

Comment on attachment 8531378 [details] [diff] [review] Centered feedback form in Loop standalone ended room conversation view. Review of attachment 8531378 [details] [diff] [review]: ----------------------------------------------------------------- This breaks the EndedConversationView - according to the ui-showcase.

Comment on attachment 8531689 [details] [diff] [review] Centered feedback form in Loop standalone ended room conversation view. Review of attachment 8531689 [details] [diff] [review]: ----------------------------------------------------------------- Looks good, r=Standard8

Nicolas, could you fill the uplift request? Not tracking because it is polishing.

Comment on attachment 8531689 [details] [diff] [review] Centered feedback form in Loop standalone ended room conversation view. Approval Request Comment [Feature/regressing bug #]: Rooms standalone feedback [User impact if declined]: merge conflicts in a shared file if we uplift more changes. [Describe test coverage new/current, TBPL]: manual testing [Risks and why]: No risk (standalone changes only). [String/UUID change made/needed]: none

Sorry, jetlag killed me.

Verified fixed in Chrome and latest nightly using the dev-server.