Closed Bug 1133662 Opened 10 years ago Closed 10 years ago

[ReadingList] Show something useful in the sidebar when the Reading List is empty

Categories

(Firefox Graveyard :: Reading List, defect, P3)

Product:
Firefox Graveyard
Component:
Reading List
Type:
defect

Tracking

(firefox38 verified, firefox39 verified)

Status:
VERIFIED FIXED
Milestone:
Firefox 39
Tracking Flags:
Tracking Status
firefox38 --- verified
firefox39 --- verified

People

(Reporter: Unfocused, Assigned: Dolske)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

Patch v.2
(deleted), patch
florian
: review+
Details | Diff | Splinter Review
If all items in the ReadingList have been removed, showing a blank sidebar may make it feel broken. Ideally we'd show some message, maybe an encouragement to add items or something like that. See also bug 1124400 comment 9, and related comments.
Flags: qe-verify+
Flags: firefox-backlog+
Michael: Thoughts on this? (If we want this for v1, we need to know the strings for it as soon as possible.)
Flags: needinfo?(mmaslaney)
What do we currently display for an empty bookmarks side panel? It would be great if we could provide a Reading List tip like "Click on the (+) icon in the URL bar for quick adding", or something similar.
Flags: needinfo?(mmaslaney)
(In reply to Michael Maslaney [:mmaslaney] (mmaslaney@mozilla.com) from comment #2) > What do we currently display for an empty bookmarks side panel? Nothing. But AFAIK it's not possible to get into that state anyway (due to standard folders that can't be removed). With history it's possible, but we just show nothing there: http://d.pr/i/15MuV With ReadingList, it's going to be a lot more common to have an empty list.
just to note that bug 1132988 is going to provide SUMO articles as default links for the reading list so the reading lists won't be empty to start. That FTU work is being tracked in bug 1132951
See Also: → https://bugzilla.mozilla.org/show_bug.cgi?id=1132988
Matej, could you review the copy for the empty Reading List sidebar? Feel free to improve on the current proposal. http://d.pr/i/1eSvw
Flags: needinfo?(matej)
(In reply to Michael Maslaney [:mmaslaney] (mmaslaney@mozilla.com) from comment #5) > Matej, could you review the copy for the empty Reading List sidebar? Feel > free to improve on the current proposal. > > http://d.pr/i/1eSvw Would this be too long? "Add articles to your Reading List to save them for later and find them easily when you need them." If so, here's a shorter option: "Add articles to your Reading List to save them for later."
Flags: needinfo?(matej)
Blocks: 1132074
(In reply to Matej Novak [:matej] from comment #6) > "Add articles to your Reading List to save them for later and find them > easily when you need them." http://d.pr/i/1fySI > "Add articles to your Reading List to save them for later." http://d.pr/i/14qj1 I think the longer one fits in fine - what do you think, Michael?
Flags: needinfo?(mmaslaney)
(In reply to Blair McBride [:Unfocused] (I don't read bugmail - needinfo? me!) from comment #7) > (In reply to Matej Novak [:matej] from comment #6) > > "Add articles to your Reading List to save them for later and find them > > easily when you need them." > > http://d.pr/i/1fySI > > > "Add articles to your Reading List to save them for later." > > http://d.pr/i/14qj1 > > I think the longer one fits in fine - what do you think, Michael? I think so, too, and it feels a little friendlier. That would be my vote. Thanks.
(In reply to Blair McBride [:Unfocused] (I don't read bugmail - needinfo? me!) from comment #7) > (In reply to Matej Novak [:matej] from comment #6) > > "Add articles to your Reading List to save them for later and find them > > easily when you need them." > > http://d.pr/i/1fySI > > > "Add articles to your Reading List to save them for later." > > http://d.pr/i/14qj1 > > I think the longer one fits in fine - what do you think, Michael? • +1 for the longer/friendlier version
Flags: needinfo?(mmaslaney)
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
Priority: -- → P3
Nominally P4 polish, but as it feels like broken UI let's call this P3.
Attached patch Patch v.1 (obsolete) (deleted) — — Splinter Review
Easy peasy.
Assignee: nobody → dolske
Attachment #8579741 - Flags: review?(florian)
Comment on attachment 8579741 [details] [diff] [review] Patch v.1 Review of attachment 8579741 [details] [diff] [review]: ----------------------------------------------------------------- I renamed the sidebar.css file to sidebar.css.inc in bug 1123525, so this patch won't apply. All the mockups linked in this bug have the placeholder text centered, I don't see anything in this patch centering the text; is this intentional?
Attachment #8579741 - Flags: feedback+
Attached patch Patch v.2 (deleted) — — Splinter Review
Merged to tip of fx-team.
Attachment #8579741 - Attachment is obsolete: true
Attachment #8579741 - Flags: review?(florian)
Attachment #8579758 - Flags: review?(florian)
Comment on attachment 8579758 [details] [diff] [review] Patch v.2 Review of attachment 8579758 [details] [diff] [review]: ----------------------------------------------------------------- Looks good :-) ::: browser/themes/shared/readinglist/sidebar.inc.css @@ +17,5 @@ > } > > +#emptyListInfo { > + padding: 3em 1em; > + text-align: center; Let's add a | cursor: default;| in here.
Attachment #8579758 - Flags: review?(florian) → review+
QA Contact: andrei.vaida
https://hg.mozilla.org/mozilla-central/rev/bb305274f262
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox39: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 39
Iteration: --- → 39.2 - 23 Mar
Blocks: 1132074
Verified fixed on Nightly 39.0a1 (2015-03-20) using Ubuntu 14.04 (x64), Windows 7 (x64) and Mac OS X 10.9.5. Please note that this is still affected by Bug 1137089, so the "Add articles to your Reading List to save them for later and [...]" text and its background are not adapting properly to high-contrast and Developer Edition themes.
Status: RESOLVED → VERIFIED
status-firefox39: fixed → verified
Verified fixed Aurora 38.0a2 (2015-03-29) as well, using Windows 7 (x64), Ubuntu 14.04 (x64) and Mac OS X 10.9.5.
status-firefox38: fixed → verified
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: