Open Bug 1580542 Opened 3 years ago Updated 4 months ago

Allow Users To Control How Many Pocket Stories Are Displayed in New Tab


(Firefox :: Pocket, enhancement, P2)





(Reporter: digitalnewslab, Unassigned, NeedInfo)




(3 files)

Attached image Fx-69-New.png
  • Firefox 69 New Tab layout displays 3 columns and 7 rows of stories, without the option for users to select how many rows are displayed.
  • Provide the option for users to choose their preferred New Tab layout.
Attached image Fx-69.png
Summary: Allow Users To Control How Pocket Stories Are Displayed in New Tab → Allow Users To Control How Many Pocket Stories Are Displayed in New Tab

@wolasi - can you give us some UX specs for this during 71 nightly for consideration for Engineering for 72? (There's a lot to consider here with today's version vs. future layouts and how we want to approach this)

Flags: needinfo?(wkonu)
Priority: -- → P2
Duplicate of this bug: 1581923

I see a control for rows preference in Developer Edition but not in Nightly. That's strange!

I think the code at lines 103-107 of AboutPreferences.jsm (dxr) may be causing this:

100  handleDiscoverySettings(sections) {
101    // Deep copy object to not modify original Sections state in store
102    let sectionsCopy = JSON.parse(JSON.stringify(sections));
103    sectionsCopy.forEach(obj => {
104      if ( === "topstories") {
105        obj.rowsPref = "";
106      }
107    });
108    return sectionsCopy;
109  }

Similar code for Highlights was removed in bug 1559536 (Fx69). What happens if these lines do not run? Does that section get the rows control just like Highlights? (Sorry, I have no idea how to test such things myself.)

Also, maxRows could be increased to 7 here:

Duplicate of this bug: 1555437
Duplicate of this bug: 1611211
Component: New Tab Page → Pocket
Duplicate of this bug: 1748607

Redirect a needinfo that is pending on an inactive user to the triage owner.
:thecount, since the bug has high priority and recent activity, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(wkonu) → needinfo?(sdowne)
You need to log in before you can comment on or make changes to this bug.