Skip to content

Conversation

AkshaySakare
Copy link

Basic splashscreen and icon added

Copy link
Contributor

@anandwana001 anandwana001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add more details in PR description and screenshot too

Comment on lines 60 to 62



Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unnecessary empty lines

modifier = Modifier.fillMaxSize(),
color = colorResource(id = R.color.app_black)
) {
showSplash()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

explore this api - androidx.core:core-splashscreen

horizontalAlignment = Alignment.CenterHorizontally,

) {
val ctx = LocalContext.current;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

variable naming could be more better


) {
val ctx = LocalContext.current;
Text(text = "Welcome", color = colorResource(id = R.color.white), fontWeight = FontWeight.Bold,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no hardcode strings


) {
val ctx = LocalContext.current;
Text(text = "Welcome", color = colorResource(id = R.color.white), fontWeight = FontWeight.Bold,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

color can be use from Color.kt file

Comment on lines 91 to 92


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto

contentDescription ="image",
alignment = Alignment.Center,

modifier = Modifier.matchParentSize()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it required?

@AkshaySakare
Copy link
Author

In " androidx.core:core-splashscreen" api need to resize icon that takes lot of time from my side ,so I decided explore more animation . New changes added lottie animation with jetpack compose. Will update more UI in next PR.
animation

Comment on lines 65 to 68
) { //Surface(modifier = Modifier.fillMaxSize(), color = colorScheme.background) {
// Lottie animation screen.
ComposeLottieScreen()
//}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we clean?

Comment on lines 75 to 77
@Composable
fun ComposeLottieScreen() {

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we have a separate file for these UI functions?

horizontalAlignment = CenterHorizontally,

) {
val sContext = LocalContext.current
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what does s stands for in sContext?

@AkshaySakare
Copy link
Author

update the changes with name and create new file for the splash-screen module

) {
val sContext = LocalContext.current
Text(
text = sContext.getString(R.string.welcome),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stringResources()

}
}

private fun isInternetAvailable(context: Context): Boolean {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

separate file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants