Closed
Bug 1074696
Opened 10 years ago
Closed 10 years ago
As a user, I should be able to copy the room url to the clipboard or email the room url from the room preview
Categories
(Hello (Loop) :: Client, enhancement, P1)
Hello (Loop)
Client
Tracking
(firefox35 verified, firefox36 verified)
backlog | Fx35+ |
People
(Reporter: standard8, Assigned: standard8)
References
()
Details
(Whiteboard: [rooms][strings])
User Story
* As a user, I should be able to copy the room url to the clipboard form the room preview, so that I can easily share the url UX: * Add the Invite someone text, implement the copy button * When the copy button is pressed, copy the string and change the text to "Copied!" Strings: * Invite someone to join you. * Copy Link * Copied! Implementation * Use the mozLoop.copyString API to copy to clipboard
Attachments
(2 files)
239.95 KB,
image/png
|
Details | |
13.78 KB,
patch
|
NiKo
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Updated•10 years ago
|
User Story: (updated)
Whiteboard: [rooms] → [rooms][strings]
Updated•10 years ago
|
User Story: (updated)
Updated•10 years ago
|
backlog: --- → Fx35+
Updated•10 years ago
|
Updated•10 years ago
|
Assignee: nobody → dmose
Updated•10 years ago
|
Assignee: dmose → nobody
Updated•10 years ago
|
Priority: -- → P1
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → standard8
Iteration: --- → 36.3
Points: --- → 2
Assignee | ||
Comment 1•10 years ago
|
||
This adds the simple bits necessary for copy and email. Although email isn't part of this bug, its just as simple and we didn't have a specific bug for it.
Attachment #8520659 -
Flags: review?(nperriault)
Assignee | ||
Updated•10 years ago
|
Summary: As a user, I should be able to copy the room url to the clipboard from the room preview → As a user, I should be able to copy the room url to the clipboard or email the room url from the room preview
Comment on attachment 8520659 [details] [diff] [review]
Allow the user to copy and email Loop room urls from the room preview
Review of attachment 8520659 [details] [diff] [review]:
-----------------------------------------------------------------
Look awesome!
::: browser/components/loop/content/js/roomViews.jsx
@@ +52,3 @@
> // Used by the UI showcase.
> roomState: this.props.roomState || storeState.roomState
> + }, storeState);
*shudder*
Attachment #8520659 -
Flags: review?(nperriault) → review+
Assignee | ||
Comment 4•10 years ago
|
||
Target Milestone: --- → mozilla36
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 6•10 years ago
|
||
Updated•10 years ago
|
status-firefox35:
--- → fixed
status-firefox36:
--- → fixed
Comment 7•10 years ago
|
||
Verified fixed FF 35b4, 36.0a2 (2014-12-18) Ubuntu 13.04
Assignee | ||
Updated•9 years ago
|
Flags: in-testsuite? → in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•