248358 - Remember how many extensions/themes should be displayed per page

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: Public Pages, enhancement, P4)

Description

[Frank Fridlund]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040614 Firefox/0.9
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040614 Firefox/0.9

Though 10 is a good default value for a list of extensions or themes for the
time being, some users may wish to display more or less per page.  Currently,
you can change the setting to a higher or lower number, and switch to different
categories without having to reset the choice.  Once the user switches products
or switches to themes/extensions, it defaults back to 10.

Reproducible: Always
Steps to Reproduce:
1. On update.mozilla.org, select Firefox
2. Click on extensions
3. Click on "All" on the left
4. Change number to display from 10 to any number and click Update
5. Click on Themes, and then click on "All"

Actual Results:  
After step 4, the number retains the value you choose.
After step 5, the number resets to 10

Expected Results:  
The number should stay the same in both steps 4 and 5

It may not be common, but some people might like longer or shorter lists.  The
easiest thing would be to set a cookie and just default to 10 if the user
doesn't want a cookie.

Comment 1

[Wolf [:wolf]]

So basically, once you change it once, have that be remembered between sections
and unless you change it again, have that stick? between sessions for for that
session-only? i'm thinking the second. Comments?

Comment 2

[Jason Chu]

I think save it for the current session only.  That seems more natural and
inobstrusive.

Comment 3

[Richard Klein]

I would like this to be cross session.  Everytime I look at the extension list a
change the number to show.

Comment 4

[Wolf [:wolf]]

Working on this.

Most likely, this'll be for the current-session only, by default.

Will see about adding a cross-session option though.

Comment 5

[Wolf [:wolf]]

OK, got a fix for this, it remembers for the session only.

I probably can extend that to be cross-session for those who want that. but this
fix doesn't include that. (That'd be part of personal preference code that
doesn't exist yet.)

Comment 6

[Wolf [:wolf]]

Fixed.