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

Reveal the keyboard background when navigation bar is visible #1236

Merged
merged 3 commits into from
Feb 9, 2024

Conversation

WhiredPlanck
Copy link
Collaborator

Pull request

Issue tracker

Fixes will automatically close the related issues

Fixes #1212 (partially)
Fixes #293 (reopened)
Fixes #519 (reopened)

Feature

Describe features of this pull request

We did the similar thing in the past (#492), but the solution is not perfect and then it was reverted.

Known issue

A little of the bottom part of the keyboard view will be masked by the navigation bar after changing theme or color scheme in place on Android 9 due to a bug from the latter. There seems to be no any reliable way to fix it.

Code of conduct

Style lint

  • make sytle-lint

Build pass

  • make debug

Manually test

  • Done

Code Review

  1. No wildcards import
  2. Manual build and test pass
  3. GitHub Action CI pass
  4. At least one contributor review and approve
  5. Merged clean without conflicts
  6. PR will be merged by rebase upstream base

Daily build

Login and download artifact at https://github.com/osfans/trime/actions

Additional Info

Before

After

refactor: slightly refactor to app theme scheme
... to meet the navigation bar immersion requirement. This will so kind take the place of `root_background`.
@WhiredPlanck WhiredPlanck merged commit 79b0268 into osfans:develop Feb 9, 2024
3 checks passed
@WhiredPlanck WhiredPlanck deleted the navbar branch February 12, 2024 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant