Add Beta tag to the Rooms standalone page

RESOLVED FIXED in Firefox 35

Status

defect
P1
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: mreavy, Unassigned)

Tracking

unspecified
mozilla37
x86_64
Windows 7
Points:
1
Bug Flags:
qe-verify -

Firefox Tracking Flags

(firefox35+ fixed, firefox36 fixed, firefox37 fixed)

Details

Attachments

(1 attachment, 2 obsolete attachments)

Since the feature is still in Beta during Fx35, we want to add the Beta ribbon to the Rooms standalone page.  We want to do this before the next update of the standalone app
backlog: --- → Fx35+
Priority: -- → P1
Minimalistic approach. Note that the issue with RWD will be addressed along bug 1104051.
Attachment #8530299 - Flags: review?(mdeboer)
Thanks, Niko!
Assignee: nobody → nperriault
Fixed regression introduced by previous patch.
Attachment #8530299 - Attachment is obsolete: true
Attachment #8530299 - Flags: review?(mdeboer)
Attachment #8530317 - Flags: review?(mdeboer)
[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 8530317 [details] [diff] [review]
Added beta tag to standalone room layout. Patch v2

I'd like us to land the remaining open Rooms bugs this week.  This just needs a review from Mike or Mark to land (whoever gets to it first).
Attachment #8530317 - Flags: review?(standard8)
Attachment #8530317 - Flags: review?(standard8) → review?(dmose)
Comment on attachment 8530317 [details] [diff] [review]
Added beta tag to standalone room layout. Patch v2

r=dmose with a comment added about what factors influenced the choice of 50px.  If it makes sense to switch to rems for scalability, I'd be ok with that too.
Attachment #8530317 - Flags: review?(dmose) → review+
I discovered an issue with that patch when resizing the browser window where the Firefox logo was hidden entirely; so here's new version of the patch fixing that situation.

Re commenting on the 50px value, I don't have a clue how to explain that choice except "because it's where it looks best"… Converting to rem wouldn't improve scalability as we're speaking svg and background here.
Attachment #8530317 - Attachment is obsolete: true
Attachment #8530317 - Flags: review?(mdeboer)
Attachment #8531738 - Flags: review?(dmose)
Comment on attachment 8531738 [details] [diff] [review]
Added beta tag to standalone room layout.

r=dmose
Attachment #8531738 - Flags: review?(dmose) → review+
https://hg.mozilla.org/integration/fx-team/rev/656951a75391
Iteration: --- → 37.1
Points: --- → 1
Target Milestone: --- → mozilla37
https://hg.mozilla.org/mozilla-central/rev/656951a75391
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Comment on attachment 8531738 [details] [diff] [review]
Added beta tag to standalone room layout.

Approval Request Comment
[Feature/regressing bug #]: Rooms standalone

[User impact if declined]: minor merge conflicts possible in future patches

[Describe test coverage new/current, TBPL]: manual test, on m-c

[Risks and why]: no risk, pure standalone 

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