Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor followup to #7335 #7347

Merged
merged 4 commits into from
Feb 25, 2021
Merged

Minor followup to #7335 #7347

merged 4 commits into from
Feb 25, 2021

Conversation

NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Feb 25, 2021


This change is Reviewable

layout.

Stuff that concerns the global/screen layout is redundant here, as Menus
are re-initialized on screen geometry updates.
@NiLuJe
Copy link
Member Author

NiLuJe commented Feb 25, 2021

@poire-z: Your analysis of the _recalculateDimen stuff appears to hold true, nothing seems to have blown up with it gone ;p.

(And, re-reading how it's used, I would agree with your assessment. Possibly a remnant from back when we didn't handle screen geometry updates sanely in the FM/FM-adjacent stuff?).

@Frenzie Frenzie added this to the 2021.03 milestone Feb 25, 2021
Copy link
Contributor

@poire-z poire-z left a comment

Choose a reason for hiding this comment

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

The cleaner the better :)

@NiLuJe NiLuJe merged commit 26b9ab2 into koreader:master Feb 25, 2021
@hius07
Copy link
Member

hius07 commented Feb 26, 2021

Minor issue in page numbers footer:
Reading statistics - Calendar view - empty day - Page 1 of 0

@NiLuJe
Copy link
Member Author

NiLuJe commented Feb 26, 2021

What do you mean by "empty day"?

@hius07
Copy link
Member

hius07 commented Feb 26, 2021

No records for that day

@hius07
Copy link
Member

hius07 commented Feb 26, 2021

1

@NiLuJe
Copy link
Member Author

NiLuJe commented Feb 26, 2021

Oh, the day view, right :).

NiLuJe added a commit that referenced this pull request Feb 26, 2021
)

By disabling the pagination buttons, and showing a specific label, like Menu (#7347 (comment))
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.

None yet

4 participants