[ReadingList] Allow adding items to the Reading List via the sidebar

NEW
Unassigned

Status

P2
normal
4 years ago
3 years ago

People

(Reporter: Unfocused, Unassigned)

Tracking

(Blocks: 1 bug)

Dependency tree / graph
Bug Flags:
firefox-backlog +
qe-verify +

Details

(Whiteboard: [reader-ui])

We want to be able to add items to the Reading List via the sidebar UI.

Undetermined as-yet whether this should only be the current tab, or it should prompt for a URL.
Micheal: Do we want to prompt for a URL here? If so, could you mock that up?
Flags: needinfo?(mmaslaney)

Updated

4 years ago
Flags: firefox-backlog+
Blocks: 1123529
Flags: qe-verify+
A URL prompt seems appropriate for the URL "Add to Reading List" button, but not the sidebar. Bit of a disconnect
Flags: needinfo?(mmaslaney)
Clarified this on IRC: Button will just add the current tab's URL. Same with the button in the URLBar.

So that keeps v1 nice and simple. We'll look into the possibility of adding arbitrary URLs (not loaded in a tab) in v2+.
Sorting out strings... Michael, can you confirm?

Button label: "Add to Reading List"
Button tooltip: "Add this page to your Reading List"

(The bookmarks star button's tooltip is "Bookmark this page")
Flags: needinfo?(mmaslaney)
Oh, I guess there's also some overlap here with bug 1131461 and https://projects.invisionapp.com/share/NK1ZBQ6SY#/screens/59725441

Although, given the sidebar isn't so obviously associated with the current tab, I'm not sure using the same exact wording is best.
+1
Flags: needinfo?(mmaslaney)

Updated

4 years ago
Blocks: 1132074
Not required for Reading List V1 Campaign Release.
No longer blocks: 1132074
Mass change of ReadingList bugs, moving to their own component. Filter bugspam on the following quote:

“Reading is to the mind what exercise is to the body.”
― Joseph Addison
Component: General → Reading List
P2 based on this being a compromise for A11Y with the URL bar "add to RL" button, which has some A11Y issues.
Priority: -- → P2

Updated

3 years ago
Blocks: 1132074
Whiteboard: [reader-ui]

Updated

3 years ago
Assignee: nobody → florian
Status: NEW → ASSIGNED
Iteration: --- → 40.2 - 27 Apr
Points: --- → 3

Updated

3 years ago
Iteration: 40.2 - 27 Apr → 40.3 - 11 May
ReadingList is no longer our priority.
Assignee: florian → nobody
Status: ASSIGNED → NEW

Updated

3 years ago
Iteration: 40.3 - 11 May → ---
(Assignee)

Updated

3 years ago
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.