WordPress.org

Making WordPress.org

Opened 3 years ago

Closed 3 years ago

#2134 closed defect (fixed)

Forums: Spam/Pending/Archived views should append ?view=all to topic/reply links

Reported by: SergeyBiryukov Owned by: coffee2code
Milestone: Priority: normal
Component: Support Forums Keywords:
Cc:

Description

Since [4225], Spam/Pending/Archived views have some context for posts (i.e. forum name, topic name, topic link).

However, if you follow the topic/reply links from those views, you can't see the actual spam/pending/archived post on the topic page, because ?view=all is missing in the URL.

Attachments (1)

meta-2134.patch (1.6 KB) - added by SergeyBiryukov 3 years ago.

Download all attachments as: .zip

Change History (10)

#2 @coffee2code
3 years ago

  • Owner set to coffee2code
  • Status changed from new to accepted

#3 @coffee2code
3 years ago

In 4232:

Support Forums, Moderators: Add class constant VIEWS to list moderator-only views.

See #2134.

#4 @coffee2code
3 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 4233:

Forums Theme: Append '?view=all' to context forum and topic links for posts listed in moderator views (e.g. spam/pending/archived).

Fixes #2134.

#6 @SergeyBiryukov
3 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Noticed a couple of things:

  • In reply to: is a link, As the topic: is not. It should be a link too for consistency.
  • Forum and topic links now have ?view=all, but individual post links in .bbp-topic-post-date and .bbp-reply-post-date do not, so you have to additionally scroll to the post you clicked on, and it might not even be on the first page.

meta-2134.patch handles the first point. Should the second point wait for #bbPress2938?

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


3 years ago

#8 @SergeyBiryukov
3 years ago

In 4770:

Support Theme: Append '?view=all' to context forum and topic links for posts listed in moderator views (e.g. spam/pending/archived).

Merges [4233] to the new theme.

Props coffee2code.
See #2134.

#9 @SergeyBiryukov
3 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

meta-2134.patch was merged in [4770], so going to close as fixed.

A new ticket can be opened for the second point of comment:6.

Note: See TracTickets for help on using tickets.