mirror of
https://github.com/go-shiori/shiori.git
synced 2026-04-25 14:35:52 +03:00
[GH-ISSUE #549] Clicking on tag shows no bookmarks in search #305
Labels
No labels
component:backend
component:builds
component:builds
component:extension
component:frontend
component:readability
database
database:mysql
database:postgres
database:sqlite
feature:ebooks
github_actions
good first issue
hacktoberfest
note:duplicate?
note:fixed?
note:out-of-scope?
os:windows
priority:high
priority:low
pull-request
resolution:as-intended
resolution:cant-reproduce
resolution:duplicate
resolution:fixed
resolution:wontfix
tag:TBD
tag:big-task
tag:help-wanted
tag:huge-data
tag:meta
tag:more-info
tag:next
tag:no-stale
tag:requires-migrations
tag:research
tag:security 🛡️
tag:stale
tag:waiting-for-assignee
type:bug
type:documentation
type:enhancement
type:meta
type:ux
user:cli
user:web
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/shiori#305
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @ryanwr on GitHub (Jan 5, 2023).
Original GitHub issue: https://github.com/go-shiori/shiori/issues/549
Data
Describe the bug / actual behavior
When clicking on a tag on a bookmark that is in a page > 1, it results in the search returning no results.
Expected behavior
Bookmarks are returned in results.
To Reproduce
Steps to reproduce the behavior:
Notes
This issue is caused because the page query parameter is not stripped when clicking on a tag on a bookmark.
If you are on page 2 the url is as follows:
/?page=2Click on tag e.g. 'webdev'.
/?page=2&search=tag%3AwebdevHowever there is only 1 bookmark and hence page 2 is empty.
Probably just needs to set
this.page = 1;here:github.com/go-shiori/shiori@a950bfec4c/internal/view/js/page/home.js (L295-L297)I can create a PR if it is an acceptable fix.
Thanks,
Ryan