Skip to content

Commit

Permalink
UI: Update check icon color in TimeTableScreen (#612)
Browse files Browse the repository at this point in the history
### TL;DR
Added color tinting to the check icon in the TimeTableScreen's Done button

### What changed?
Added a `ColorFilter.tint()` to the check icon image in the TimeTableScreen, using the foreground color derived from the current theme color

### How to test?
1. Navigate to the TimeTableScreen
2. Verify that the check icon in the Done button matches the theme's foreground color
3. Test with different theme colors to ensure proper contrast

### Why make this change?
To ensure the check icon maintains proper visibility and consistency with the app's theme, particularly when different theme colors are applied
  • Loading branch information
ksharma-xyz authored Feb 13, 2025
1 parent 071d725 commit 80832f9
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ import xyz.ksharma.krail.taj.components.TitleBar
import xyz.ksharma.krail.taj.hexToComposeColor
import xyz.ksharma.krail.taj.modifier.klickable
import xyz.ksharma.krail.taj.theme.KrailTheme
import xyz.ksharma.krail.taj.theme.getForegroundColor
import xyz.ksharma.krail.trip.planner.ui.components.ActionData
import xyz.ksharma.krail.trip.planner.ui.components.ErrorMessage
import xyz.ksharma.krail.trip.planner.ui.components.JourneyCard
Expand Down Expand Up @@ -257,6 +258,7 @@ fun TimeTableScreen(
Image(
painter = painterResource(Res.drawable.ic_check),
contentDescription = null,
colorFilter = ColorFilter.tint(color = getForegroundColor(themeColor)),
modifier = Modifier.size(20.dp),
)
Text("Done", modifier = Modifier.padding(start = 4.dp))
Expand Down

0 comments on commit 80832f9

Please sign in to comment.