-
-
Notifications
You must be signed in to change notification settings - Fork 530
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 in-app screensaver #2539
Add in-app screensaver #2539
Conversation
0d45f87
to
0fa64f4
Compare
timer?.cancel() | ||
|
||
// Hide when interacted with allowed cancelation or when disabled | ||
if (_visible.value && (canCancel || !enabled || activityPaused)) { |
Check warning
Code scanning / detekt
Complex conditions should be simplified and extracted into well-named methods if necessary.
|
||
AnimatedVisibility( | ||
visible = visible, | ||
enter = fadeIn(tween(1_000)), |
Check warning
Code scanning / detekt
Report magic numbers. Magic number is a numeric literal that is not defined as a constant and hence it's unclear what the purpose of this number is. It's better to declare such numbers as constants and give them a proper name. By default, -1, 0, 1, and 2 are not considered to be magic numbers.
AnimatedVisibility( | ||
visible = visible, | ||
enter = fadeIn(tween(1_000)), | ||
exit = fadeOut(tween(1_000)), |
Check warning
Code scanning / detekt
Report magic numbers. Magic number is a numeric literal that is not defined as a constant and hence it's unclear what the purpose of this number is. It's better to declare such numbers as constants and give them a proper name. By default, -1, 0, 1, and 2 are not considered to be magic numbers.
Changes
getQuantityString
utilityIssues