/search page drops sort and t parameters from next link #204

Open
opened 1 year ago by jbosboom · 1 comments

Steps to reproduce:

  1. Perform a search using non-default options for "sorted by:" and "links from:", bringing you to a page like https://teddit.net/r/baseball/search?q=baseball&restrict_sr=on&nsfw=on&sort=top&t=month .
  2. Scroll to the bottom of the page. Observe that the "next ›" link does not have a sort or t parameter, so clicking that link results in a page with no results.
  3. Manually edit the URL to add the missing parameters and click the link. (There are results.)
  4. Scroll to the bottom of the page. Observe that the "next ›" link is again missing the sort and t parameters.

(Similar to #194.)

Steps to reproduce: 1. Perform a search using non-default options for "sorted by:" and "links from:", bringing you to a page like https://teddit.net/r/baseball/search?q=baseball&restrict_sr=on&nsfw=on&sort=top&t=month . 2. Scroll to the bottom of the page. Observe that the "next ›" link does not have a `sort` or `t` parameter, so clicking that link results in a page with no results. 3. Manually edit the URL to add the missing parameters and click the link. (There are results.) 4. Scroll to the bottom of the page. Observe that the "next ›" link is again missing the `sort` and `t` parameters. (Similar to [#194](https://codeberg.org/teddit/teddit/issues/194).)
teddit added the
bug
label 1 year ago
Owner

Now both prev and next link include sort and t parameters. 7c7709e74f The search functionality doesn't still work perfectly, especially when prev link is used, but it's better now.

Now both prev and next link include `sort` and `t` parameters. 7c7709e74f The search functionality doesn't still work perfectly, especially when prev link is used, but it's better now.
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: teddit/teddit#204
Loading…
There is no content yet.