fix: keep geohash field and Next button visible when keyboard opens#492
Merged
torlando-tech merged 2 commits intomainfrom Feb 18, 2026
Merged
fix: keep geohash field and Next button visible when keyboard opens#492torlando-tech merged 2 commits intomainfrom
torlando-tech merged 2 commits intomainfrom
Conversation
On the offline map download location step, the geohash text field was hidden behind the soft keyboard on shorter displays. Adds imePadding() and restructures the layout so the scrollable form fields shrink above the keyboard while the Next button stays pinned at the bottom. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
Greptile SummaryRestructured the Key changes:
The implementation follows Android best practices for handling IME insets in Compose. The layout structure now ensures that when the keyboard appears, the geohash field and Next button remain visible while still allowing the form to scroll to access other fields like latitude/longitude. Confidence Score: 5/5
Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Root Column<br/>fillMaxSize + imePadding + padding] --> B[Inner Column<br/>weight 1f + fillMaxWidth + verticalScroll]
A --> C[Spacer<br/>height 16.dp]
A --> D[Next Button<br/>fillMaxWidth - pinned at bottom]
B --> E[Warning Banner]
B --> F[Instructions Text]
B --> G[Current Location Button]
B --> H[Lat/Lon TextFields]
B --> I[Address Search conditional]
B --> J[Geohash TextField]
B --> K[Location Confirmation Card]
style A fill:#e1f5ff
style B fill:#fff9c4
style D fill:#c8e6c9
style J fill:#ffccbc
Last reviewed commit: f398999 |
…ard-scroll # Conflicts: # app/src/main/java/com/lxmf/messenger/ui/screens/offlinemaps/OfflineMapDownloadScreen.kt
Contributor
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
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
imePadding()to the location selection step so the content shrinks above the soft keyboardTest plan
🤖 Generated with Claude Code