Design/UX #4220
closed
Fix 'Number of Posts' display option (Forum display settings)
Added by Samantha Raddatz over 9 years ago.
Updated over 9 years ago.
Category name:
Group Forums
Description
Changing the ‘Number of Posts’ option made no difference; the number of posts per page stayed the same.
Fix so that changing the number of posts per page option works as stated.
- Status changed from New to Reporter Feedback
- Assignee changed from Boone Gorges to Samantha Raddatz
Hi Sam,
Do you have the "Thread Display Mode" set to "Threaded"? If so, the "Number of Posts to Show Per Page" option is not applicable. If you select either of the "Linear" options, this should work as expected.
Try this out on cdev and let me know if it works for you.
- Category name changed from User Experience to Group Forums
Can we improve the UX here? The explanatory text about Threaded + Posts Per Page is pretty good, but I think people don't read that stuff. How about some JS or whatever that greys out the whole "per page" setting section when Threaded is selected above?
- Target version set to Future release
Let's move into a minor release if/when we decide on any necessary UX changes (assuming that this is more a documentation issue than a real bug).
How about some JS or whatever that greys out the whole "per page" setting section when Threaded is selected above?
For sure. Will do this.
The explanatory text about Threaded + Posts Per Page is pretty good, but I think people don't read that stuff.
See #4219.
Yes, it does work as you state, Ray.
And no, I didn't read the text, so Boone's assumption about no one reading that text is likely the case!
I agree that greying out/making the 2nd dropdown unclickable when Threaded is chosen would be helpful. Additionally, having text appear only when Threaded is chosen would be helpful as well: "This option is only available for linear display modes."
Sorry for the multiple-related tickets for some of this stuff -- dividing up the report for tickets proved to be harder than I expected!
- Target version changed from Future release to 1.9
Just made some updates to this.
Here's an animated GIF for the "Settings > Forums" section:
When "Thread Display Mode" is set to "Threaded", the "Number of Posts to Show Per Page" option is now disabled and a notice appears.
Let me know if any changes should be made before I add the code.
I'll remove the explanatory text in a later revision.
Looks good to me! Thanks Ray.
- Status changed from Reporter Feedback to Resolved
- Assignee changed from Samantha Raddatz to Raymond Hoh
Fixed in commit 63f6f4c.
This is set for 1.9, but we can move it back to 1.8.6 if we want.
Also available in: Atom
PDF