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

🐛 Add row weights to the fix top bar icons being hidden when text is too big #3669

Merged
merged 4 commits into from
Dec 20, 2024
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
🐛 Add Row weights to the top bar
  • Loading branch information
dubdabasoduba committed Dec 20, 2024
commit 6728efea46a8399cf97a96d8792939e9ea2fd9bb
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ fun TopScreenSection(
verticalAlignment = Alignment.CenterVertically,
horizontalArrangement = Arrangement.SpaceBetween,
) {
Row(verticalAlignment = Alignment.CenterVertically) {
Row(verticalAlignment = Alignment.CenterVertically, modifier = Modifier.weight(4f)) {
Icon(
when (toolBarHomeNavigation) {
ToolBarHomeNavigation.OPEN_DRAWER -> Icons.Filled.Menu
Expand All @@ -169,7 +169,7 @@ fun TopScreenSection(

Row(
verticalAlignment = Alignment.CenterVertically,
modifier = Modifier.padding(start = 8.dp),
modifier = Modifier.padding(start = 8.dp).weight(1f),
) {
SetupToolbarIcons(
menuIcons = topScreenSection?.menuIcons,
Expand Down Expand Up @@ -412,7 +412,7 @@ private fun RenderMenuIcon(
@Composable
fun TopScreenSectionWithFilterItemOverNinetyNinePreview() {
TopScreenSection(
title = "All Clients",
title = "All Clients All Clients All Clients All Clients All Clients",
searchQuery = SearchQuery("Eddy"),
filteredRecordsCount = 120,
onSearchTextChanged = { _, _ -> },
Expand Down
Loading