Skip to content

feat(Show Originals): Allow filtering blog search/tagged/filtered-by-type#1964

Draft
marcustyphoon wants to merge 11 commits intoAprilSylph:masterfrom
marcustyphoon:show-originals-peepr-search-tagged
Draft

feat(Show Originals): Allow filtering blog search/tagged/filtered-by-type#1964
marcustyphoon wants to merge 11 commits intoAprilSylph:masterfrom
marcustyphoon:show-originals-peepr-search-tagged

Conversation

@marcustyphoon
Copy link
Copy Markdown
Collaborator

@marcustyphoon marcustyphoon commented Nov 1, 2025

Description

Currently, Show Originals has no effect if you search or filter a specific blog's posts in Peepr. This allows one to enable Show Originals in this case.

Internally, a separate peeprFiltered location is added to the code, as I don't think it makes sense to apply the remembered mode for viewing a specific blog when searching. This mode always defaults to off and is never remembered. The location is defined as "any peepr timeline that's not the main peepr posts view" so that exhaustive timeline id utils for the various options aren't necessary.

Somewhat related to, merge conflicts with, and is probably about as niche as #1378. (By which I mean I want both of them, but who knows if anyone else does.)

Testing steps

Open a blog in peepr and turn Show Originals on on said blog.

Confirm that Show Originals shows a default-off but functional control element on that blog when:

  • filtering by post type (nothing in search bar)
  • searching for something
  • searching for something and [filtering by post type or not, filtering by original posts or not, and viewing top posts first or not]; all 8 combinations should work
  • viewing posts with a certain tag
  • viewing posts with a certain tag and filtering by post type

Clear the search and confirm that Show Originals goes back to being on, as it remembered your preference for viewing blogs in peepr with no filter.

Add the blog in question to the "always show posts from these blogs" list and confirm that Show Originals appears disabled both with and without a filter.

@marcustyphoon
Copy link
Copy Markdown
Collaborator Author

Related: #1985.

@marcustyphoon marcustyphoon marked this pull request as draft November 12, 2025 04:27
@marcustyphoon marcustyphoon self-assigned this Nov 12, 2025
@marcustyphoon marcustyphoon marked this pull request as ready for review November 13, 2025 08:36
@marcustyphoon marcustyphoon removed their assignment Nov 13, 2025
@marcustyphoon
Copy link
Copy Markdown
Collaborator Author

I don't remember this PR well; I'll have to look back into it. Anyway, I assume that because I made it as nondraft it should be fully functional, but as it doesn't have a description written it should be draft.

@marcustyphoon marcustyphoon marked this pull request as draft March 5, 2026 15:20
@marcustyphoon marcustyphoon self-assigned this Mar 5, 2026
@marcustyphoon marcustyphoon marked this pull request as ready for review March 6, 2026 15:14
@marcustyphoon marcustyphoon removed their assignment Mar 6, 2026
@marcustyphoon marcustyphoon marked this pull request as draft April 2, 2026 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant