Move reading screen menu toolbar to the bottom#3687
Open
kkebb wants to merge 4 commits into
Open
Conversation
Relocate the PagerActivity action bar from the top to the bottom of the screen so navigation, search, and settings are within easier thumb reach. - Group the audio bar and menu toolbar in a bottom-anchored container, with the menu toolbar as the bottom-most element. - Apply the bottom navigation-bar/cutout inset to the toolbar instead of the top status-bar inset. - Slide the whole bottom container down to hide it (instead of the toolbar sliding up and the audio bar sliding down separately). - Drop the audio bar's bottom inset since the toolbar now sits below it. Co-authored-by: Cursor <cursoragent@cursor.com>
This was referenced Jun 16, 2026
Apply IME insets to the bottom bar container so the audio bar and menu toolbar move above the soft keyboard when the SearchView is expanded. Co-authored-by: Cursor <cursoragent@cursor.com>
Hide the audio bar while SearchView is expanded and apply IME insets to the menu toolbar only, instead of lifting the entire bottom bar stack. Co-authored-by: Cursor <cursoragent@cursor.com>
Move system bar and cutout padding from the transparent bottom container onto toolbar_area so its background fills the gesture-nav zone, while still merging IME lift for search. Co-authored-by: Cursor <cursoragent@cursor.com>
8 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Relocates the reading screen (
PagerActivity) action bar from the top to the bottom of the screen, so navigation, search, and settings sit within easier thumb reach during reading.toolbar_area(which has a solid background), so the gesture-nav zone is filled with no transparent gap showing the mushaf underneath.PR stack & overlap
This PR is one of three related UX improvements. It does not share any files with #3688 — they touch different screens:
PagerActivity)QuranActivity)Both bottom PRs use similar patterns (bottom chrome, inset handling, search keyboard behavior) but on different activities and layout files. They can merge in either order.
Changes
app/.../ui/PagerActivity.kt—bottomBarAreareference, bottom inset + IME handling ontoolbar_area, unifiedanimateToolBar, search expand hides audio bar.app/.../res/layout/quran_page_activity.xml— wrap audio bar + toolbar in a bottom-anchored vertical container.feature/audiobar/.../AudioBarWrapper.kt— only apply horizontal insets now (toolbar handles bottom inset).Related
Test plan
PagerActivity; confirm the menu toolbar appears at the bottom with title/up/search/overflow.SearchViewwork correctly from the bottom position.Test results
:appunit test suite passing (0 failures, 0 errors)Also verified on an Android emulator (API 36, Pixel 6): reading toolbar renders at the bottom with audio bar stacked above it when playback is active; bottom gap fix confirmed (
toolbar_areaextends to screen bottom).Made with Cursor