Skip to content

[Filechooser] 0% > 'on hold'#12037

Merged
Frenzie merged 1 commit into
koreader:masterfrom
Commodore64user:illegal-move
Jun 24, 2024
Merged

[Filechooser] 0% > 'on hold'#12037
Frenzie merged 1 commit into
koreader:masterfrom
Commodore64user:illegal-move

Conversation

@Commodore64user

@Commodore64user Commodore64user commented Jun 16, 2024

Copy link
Copy Markdown
Member

This is a minor annoyance of mine, since the introduction of "sort_percent" (#11592) 0% and on hold marked books intermingle with one another, which was not the original intention (in my mind at least).

As a remainder, "sort_percent" is a local variable used exclusively to organise books in only this sorting mode (percent – unopened – finished last), so should not affect anyone or anything else.


This change is Reviewable

@Frenzie Frenzie left a comment

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.

Fine by me

@Frenzie Frenzie added this to the 2024.06 milestone Jun 16, 2024
@Frenzie

Frenzie commented Jun 17, 2024

Copy link
Copy Markdown
Member

@hasezoey and/or @hius07 any comments?

@hasezoey hasezoey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

sure, i dont use "on hold" personally, but i can see that "0% == on hold" (intermingling) could be annoying.

@Commodore64user

Copy link
Copy Markdown
Member Author

Should we assume that @hius07 has no objections?

@Frenzie Frenzie merged commit 799275d into koreader:master Jun 24, 2024
@Commodore64user Commodore64user deleted the illegal-move branch June 29, 2024 15:34
Commodore64user added a commit to Commodore64user/KOReader_fork that referenced this pull request Aug 2, 2024
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.

3 participants