Closed Bug 1141128 Opened 10 years ago Closed 10 years ago

Add margin to the left side of the remote video container in the standalone client

Categories

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

defect
Points:
1

Tracking

(firefox39 verified)

VERIFIED FIXED
mozilla39
Iteration:
39.2 - 23 Mar
Tracking Status
firefox39 --- verified

People

(Reporter: mikedeboer, Assigned: mikedeboer)

References

Details

(Whiteboard: [screensharing][UX bug])

Attachments

(2 files)

During the Window/ tabsharing feature review the issue was raised that the remote video stream hugs the left side of the tab/ window too much. Sevaan noted that adding a margin/ padding if 10px would make it look much better.
Flags: qe-verify+
Flags: firefox-backlog+
Attached image 10px Left Margin
Attachment #8575315 - Attachment description: bug-1141128-leftMargin.patch → Patch v1: add a margin of 10px to the left side of the remote video stream
Assignee: nobody → mdeboer
Status: NEW → ASSIGNED
Iteration: --- → 38.3 - 23 Feb
Points: --- → 1
Per Gavin's request - just updating bugs that carried over before they were 100% resolved to the next iteration.
Iteration: 38.3 - 23 Feb → 39.2 - 23 Mar
Rank: 25
Whiteboard: [screensharing][UX bug]
Comment on attachment 8575315 [details] [diff] [review] Patch v1: add a margin of 10px to the left side of the remote video stream Review of attachment 8575315 [details] [diff] [review]: ----------------------------------------------------------------- Looks good.
Attachment #8575315 - Flags: review?(standard8) → review+
Target Milestone: mozilla38 → ---
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
QA Contact: bogdan.maris
I tested this fix across platforms (Windows 7 64-bit, Mac OS X 10.9.5 and Ubuntu 14.04 32-bit) on latest Nightly. I noticed that the padding only applies when the sharing is active, not before entering the conversation and after leaving it. Is this the intended behavior?
Flags: needinfo?(mdeboer)
Hi Bogdan, I'm afraid this bug is not yet verifiable, because this fix is only available when the standalone client is updated on 'hello.firefox.com'. So this is not part of the UI inside Firefox. That said, I think it'd be good leave a note in the bugs that went into the standalone client once an update is released, so that the verification process can be kicked off from then on.
Flags: needinfo?(mdeboer)
Mark, please see comment 9. Is this feasible with the current release automation? Or can I file a bug to get this done at some point?
Flags: needinfo?(standard8)
(In reply to Mike de Boer [:mikedeboer] from comment #9) > Hi Bogdan, I'm afraid this bug is not yet verifiable, because this fix is > only available when the standalone client is updated on 'hello.firefox.com'. > So this is not part of the UI inside Firefox. This should, however, be verifiable on the development server - set your loop.server pref to "https://loop-dev.stage.mozaws.net/v0" (I recommend doing this in a separate profile). > That said, I think it'd be good leave a note in the bugs that went into the > standalone client once an update is released, so that the verification > process can be kicked off from then on. I think generally I'd prefer verification against the development servers - as then we can find issues before it hits production. Generally it hits development within an hour of the bug being marked as "fixed". That said, it might also be good to see if we can automatically comment on the bug for when it goes to production anyway. That way we can have tracking for future, to know exactly when a bug shipped. Its conceivable that we could roll this into automation that ops are already working on. Getting bug 1050295 would probably also help this work. Might be worth thinking a little bit more about the exact flow that we want before we file bugs though.
Flags: needinfo?(standard8)
(In reply to Mark Banner (:standard8) from comment #11) > (In reply to Mike de Boer [:mikedeboer] from comment #9) > > Hi Bogdan, I'm afraid this bug is not yet verifiable, because this fix is > > only available when the standalone client is updated on 'hello.firefox.com'. > > So this is not part of the UI inside Firefox. > > This should, however, be verifiable on the development server - set your > loop.server pref to "https://loop-dev.stage.mozaws.net/v0" (I recommend > doing this in a separate profile). > > > That said, I think it'd be good leave a note in the bugs that went into the > > standalone client once an update is released, so that the verification > > process can be kicked off from then on. Thanks Mark, I changed the value of loop.server and I verified that this issue is fixed on Windows 8.1 64-bit, Mac OS X 10.9.5 and Ubuntu 14.04 32-bit using latest Nightly.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: