Closed Bug 1158883 Opened 9 years ago Closed 9 years ago

Update Pocket "Page Saved" panel UI to match the final spec

Categories

(Firefox :: Pocket, defect)

defect
Not set
normal
Points:
5

Tracking

()

RESOLVED INVALID
Tracking Status
firefox40 --- affected

People

(Reporter: jaws, Assigned: jaws)

References

Details

Attachments

(2 files)

      No description provided.
Summary: Update Pocket panel UI to match the final spec → Update Pocket "Page Saved" panel UI to match the final spec
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Iteration: --- → 40.2 - 27 Apr
Flags: qe-verify?
Flags: firefox-backlog+
Attached patch PatchSplinter Review
This patch implements most of the spec attached to this bug.

It doesn't implement the tagging styles or the attachment of the Add button with the text field next to it. It also doesn't implement the suggested tags area.
Attachment #8598235 - Flags: review?(florian)
Points: --- → 5
Flags: qe-verify? → qe-verify+
Updated Specs for Pocket Integration and flow:
http://invis.io/4E2TPD4YD
Comment on attachment 8598235 [details] [diff] [review]
Patch

Review of attachment 8598235 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/base/content/browser-pocket.properties
@@ +14,5 @@
>  pocket-account-question  = Already have an account?
>  pocket-login-now         = Log in now
> +pocket-page-saved-header = Saved to Pocket
> +pocket-open-pocket       = View List
> +pocket-next-steps-splitter = |

Is this splitter something that really needs to be localizable?
Attachment #8598235 - Flags: review?(florian) → review+
Iteration: 40.2 - 27 Apr → 40.3 - 11 May
QA Contact: andrei.vaida
This bug is closed in favor of bug 1159744.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → INVALID
Iteration: 40.3 - 11 May → ---
Flags: qe-verify+
QA Contact: andrei.vaida
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: