-
-
Notifications
You must be signed in to change notification settings - Fork 88
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
Error message hard to read #938 #985
base: master
Are you sure you want to change the base?
Error message hard to read #938 #985
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally when making changes to the user interface you could add a screenshot or screen recording of the effect of the change
@@ -15,6 +15,7 @@ class TreeTrackerActivity : ComponentActivity() { | |||
private val languageSwitcher: LanguageSwitcher by inject() | |||
private val viewModelFactory: TreeTrackerViewModelFactory by inject() | |||
|
|||
//hello this is the first change i have made |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need to add this comment. Github automatically the lines and file you made changes to.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah this was on the first commit, i was trying learn how deal with github.
@@ -281,7 +281,7 @@ fun CustomSnackbar( | |||
Text( | |||
text = data.message, | |||
style = CustomTheme.typography.regular, | |||
color = CustomTheme.textColors.darkText | |||
color = CustomTheme.textColors.lightText |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good.
I'm not sure this issue was talking about the button text. I think it was referring to the black error text showing in the red box. I think this should be decided by design before committing to a change though. @EngrTaofeek can you confirm with design on this task please? |
@Elforama Yes, this changing the black error text in the red box as it shown below: |
Fixes #938
-Change the Text color to White
-add an example for the User