Skip to content

File browser: filter by status#13503

Merged
hius07 merged 7 commits into
koreader:masterfrom
hius07:filter-by-status
Apr 5, 2025
Merged

File browser: filter by status#13503
hius07 merged 7 commits into
koreader:masterfrom
hius07:filter-by-status

Conversation

@hius07

@hius07 hius07 commented Apr 2, 2025

Copy link
Copy Markdown
Member

1

2


This change is Reviewable

@hius07

hius07 commented Apr 2, 2025

Copy link
Copy Markdown
Member Author

Usage hint: long-pressing on the "Book status: ..." menu item resets the filter to "all".

"All" does not check (and fetch) book statuses.
Reminder: book statuses are cached in the BookList module.

@poire-z

poire-z commented Apr 2, 2025

Copy link
Copy Markdown
Contributor

(Not a user of this feature, no strong opinion, just a little feeling that showing some status or some others is more of a one-shot decision for the current browsing session, than a full sticky filebrowser setting, so more something I would expect from the plus menu. But in History, we have it in the hamburger menu, and it sticks. Also probably more complicated to do it from elsewhere than just replacing show_finished.)

@hius07

hius07 commented Apr 2, 2025

Copy link
Copy Markdown
Member Author

I thought about the Plus menu.
(1) +5 new buttons, overloaded popup dialog
(2) no way to have a combined filter, like "reading" + "on hold"
(3) the same number of gestures to reset the filter: (tap on the Plus icon + tap on the All button) vs (call the upper menu + long-press on the Book status item)
(4) duplicated "Show finished" menu item (actually, sticky filter by status)
(5) more lines of code

@hius07

hius07 commented Apr 2, 2025

Copy link
Copy Markdown
Member Author

In History and Collections the popup dialog is the only way to call actions (I thought about the upper touch menu, looks rather complicated).
In History filter by status sticks (there was a request for that), in Collections doesn't.

-- maintain a reference to menu_container
self.menu_container = menu_container
if not do_not_show then
if do_not_show ~= true then -- Menu search does not open the menu when called by a gesture

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Although this partially solves #13507 (currently only fixing FM but not reader), shouldn’t this be fixing the fact that random things are being passed as arguments instead?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree, let's discuss and fix it separately.

@hius07 hius07 merged commit 7e12cab into koreader:master Apr 5, 2025
@hius07 hius07 deleted the filter-by-status branch April 5, 2025 07:19
@hius07 hius07 mentioned this pull request Apr 5, 2025
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
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.

4 participants