Making WordPress.org

Opened 2 years ago

Closed 2 years ago

#7431 closed enhancement (fixed)

Remove Support Thread pagination from top of the page.

Reported by: dufresnesteven's profile dufresnesteven Owned by: dufresnesteven's profile dufresnesteven
Milestone: Priority: normal
Component: Support Forums Keywords: has-patch dev-reviewed
Cc:

Description

When viewing a plugin's support threads, we show the pagination control on the top of the page above the table. This is visually distracting and I don't understand what use case this is catering to.

We should be focusing on providing the most relevant information in front of the user, the threads in the table. This undermines that goal by giving them a shortcut to go somewhere else.

Attachments (2)

Screenshot 2024-02-02 at 10.25.30 AM.png (492.2 KB) - added by dufresnesteven 2 years ago.
7431.diff (47.6 KB) - added by dufresnesteven 2 years ago.
I agree, CSS appears to be the only way.

Download all attachments as: .zip

Change History (9)

This ticket was mentioned in Slack in #forums by dufresnesteven. View the logs.


2 years ago

#2 @bcworkz
2 years ago

I've no need for the top nav element in plugin support, however I use the same a lot in my own profile for accessing older topics I've participated in. It seems these topic lists are all based on the same template so if it goes away in plugin support it'd likely also go away elsewhere? We could simply hide the element with CSS so it doesn't clutter up areas where it's not desired while still remaining in other areas where it's useful. All without needing to change the underlying template.

#3 @dufresnesteven
2 years ago

Okay, thanks for that context. I'll look to filter it out of plugin/theme forums as the ideal first step.

@dufresnesteven
2 years ago

I agree, CSS appears to be the only way.

#4 @dufresnesteven
2 years ago

I've added a patch that hides the top pagination using CSS for all the forums. It doesn't hide it for any tables within the profile as that uses a different CSS structure. I'll do more testing before merging.

#6 @renyot
2 years ago

  • Keywords has-patch dev-reviewed added
  • Type changed from defect (bug) to enhancement

#7 @dufresnesteven
2 years ago

  • Owner set to dufresnesteven
  • Resolution set to fixed
  • Status changed from new to closed

In 13178:

Support: Remove top pagination from threads, excluding profile view.

Props: bcworkz, renyot.

Closes: #7431

Note: See TracTickets for help on using tickets.