Closed
Bug 1211563
Opened 9 years ago
Closed 9 years ago
Invite buttons should be underneath "Invite a friend" text in popped-out view
Categories
(Hello (Loop) :: Client, defect)
Hello (Loop)
Client
Tracking
(firefox43 verified, firefox44 verified)
People
(Reporter: sevaan, Assigned: Mardak)
References
Details
Attachments
(4 files)
166.57 KB,
image/png
|
Details | |
1.65 KB,
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
162.43 KB,
image/png
|
sevaan
:
ui-review+
|
Details |
1.70 KB,
patch
|
lizzard
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
When popping out the conversation window, the invitation buttons are separated by too much space from the "Invite a friend to join you" text.
The buttons should be underneath the buttons in the same manner of spacing as it looked when the conversation window is docked.
Assignee | ||
Comment 1•9 years ago
|
||
Assignee | ||
Comment 2•9 years ago
|
||
Attachment #8670000 -
Flags: ui-review?(sfranks)
Assignee | ||
Updated•9 years ago
|
Points: --- → 2
Reporter | ||
Updated•9 years ago
|
Attachment #8670000 -
Flags: ui-review?(sfranks) → ui-review+
Comment 3•9 years ago
|
||
Comment on attachment 8669999 [details] [diff] [review]
v1
Review of attachment 8669999 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good. r=Standard8
Attachment #8669999 -
Flags: review?(standard8) → review+
Assignee | ||
Comment 4•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/5e57563087b61888fb2e1551ab4a3fc4fb2ebecf
Bug 1211563 - Invite buttons should be underneath "Invite a friend" text in popped-out view [r=Standard8]
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox44:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
Assignee | ||
Comment 6•9 years ago
|
||
Approval Request Comment
[Feature/regressing bug #]: Hello Visual refresh
[User impact if declined]: Buttons can be separated from the related header text on large windows
[Describe test coverage new/current, TreeHerder]: N/a, css only
[Risks and why]: Low - css only
[String/UUID change made/needed]: None
I'll land this after bug 1184924 is uplifted.
Assignee | ||
Comment 7•9 years ago
|
||
Comment on attachment 8670939 [details] [diff] [review]
for aurora (Mardak will land)
Approval Request Comment
[Feature/regressing bug #]: Hello Visual refresh
[User impact if declined]: Buttons can be separated from the related header text on large windows
[Describe test coverage new/current, TreeHerder]: N/a, css only
[Risks and why]: Low - css only
[String/UUID change made/needed]: None
I'll land this after bug 1184924 is uplifted.
Attachment #8670939 -
Flags: approval-mozilla-aurora?
Assignee | ||
Updated•9 years ago
|
Iteration: --- → 44.2 - Oct 19
Comment 8•9 years ago
|
||
Comment on attachment 8670939 [details] [diff] [review]
for aurora (Mardak will land)
Minor css tweaks, ok to uplift to aurora
Attachment #8670939 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 9•9 years ago
|
||
http://hg.mozilla.org/releases/mozilla-aurora/rev/eeef206961d3
Bug 1211563 - Invite buttons should be underneath "Invite a friend" text in popped-out view [r=Standard8, a=lizzard]
status-firefox43:
--- → fixed
Updated•9 years ago
|
Flags: qe-verify+
QA Contact: bogdan.maris
Comment 10•9 years ago
|
||
Verified that the buttons are underneath 'Invite a fiend' string just as shown in attachment using Firefox Developer Edition 43.0a2 and latest Nightly 44.0a1 across platforms (Windows 7 64-bit, Windows 10 64-bit, Mac OS X 10.11 and Ubuntu 14.04 32-bit).
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•