Sulkta-OSS/element-x-ada
3
0
Fork 0
Code Issues 1 Pull requests 10 Projects Releases Packages Wiki Activity Actions

Ensure that navigation using keyboard is not broken on the room list.

Browse source 
Workaround https://issuetracker.google.com/issues/436432313
This commit is contained in:
Benoit Marty 2025-08-05 12:00:12 +02:00
parent eb33c8d864
commit a7ea4d01be
1 changed files with 7 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

9
features/home/impl/src/main/kotlin/io/element/android/features/home/impl/HomeView.kt
View file

@ -240,14 +240,19 @@ private fun HomeScaffold(
contentPadding = PaddingValues( contentPadding = PaddingValues(
// FAB height is 56dp, bottom padding is 16dp, we add 8dp as extra margin -> 56+16+8 = 80, // FAB height is 56dp, bottom padding is 16dp, we add 8dp as extra margin -> 56+16+8 = 80,
// and include provided bottom padding // and include provided bottom padding
bottom = 80.dp + padding.calculateBottomPadding(), // Disable contentPadding due to navigation issue using the keyboard
top = padding.calculateTopPadding() // See https://issuetracker.google.com/issues/436432313
bottom = 80.dp, // + padding.calculateBottomPadding(),
//top = padding.calculateTopPadding()
), ),
modifier = Modifier modifier = Modifier
.padding( .padding(
PaddingValues( PaddingValues(
start = padding.calculateStartPadding(LocalLayoutDirection.current), start = padding.calculateStartPadding(LocalLayoutDirection.current),
end = padding.calculateEndPadding(LocalLayoutDirection.current), end = padding.calculateEndPadding(LocalLayoutDirection.current),
// Remove these two lines once https://issuetracker.google.com/issues/436432313 has been fixed
bottom = padding.calculateBottomPadding(),
top = padding.calculateTopPadding()
) )
) )
.consumeWindowInsets(padding) .consumeWindowInsets(padding)