Fabio/Auxio
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

home: disable progress indicator on home

Browse source 
Likely consuming too much CPU given the frequency of updates
This commit is contained in:
Alexander Capehart 2024-06-22 12:55:02 -06:00
parent 043bc22eea
commit 5f73201c9c
No known key found for this signature in database
GPG key ID: 37DBE3621FE9AD47
3 changed files with 7 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
app/src/main/java/org/oxycblt/auxio/home/HomeFragment.kt
View file

@ -421,7 +421,7 @@ class HomeFragment :
when (error) { when (error) {
is NoAudioPermissionException -> { is NoAudioPermissionException -> {
logD("Showing permission prompt") logD("Showing permission prompt")
binding.homeIndexingStatus.text = context.getString(R.string.err_no_perms) binding.homeIndexingStatus.setText(R.string.err_no_perms)
// Configure the action to act as a permission launcher. // Configure the action to act as a permission launcher.
binding.homeIndexingTry.apply { binding.homeIndexingTry.apply {
text = context.getString(R.string.lbl_grant) text = context.getString(R.string.lbl_grant)
@ -436,7 +436,7 @@ class HomeFragment :
} }
is NoMusicException -> { is NoMusicException -> {
logD("Showing no music error") logD("Showing no music error")
binding.homeIndexingStatus.text = context.getString(R.string.err_no_music) binding.homeIndexingStatus.setText(R.string.err_no_music)
// Configure the action to act as a reload trigger. // Configure the action to act as a reload trigger.
binding.homeIndexingTry.apply { binding.homeIndexingTry.apply {
visibility = View.VISIBLE visibility = View.VISIBLE
@ -447,7 +447,7 @@ class HomeFragment :
} }
else -> { else -> {
logD("Showing generic error") logD("Showing generic error")
binding.homeIndexingStatus.text = context.getString(R.string.err_index_failed) binding.homeIndexingStatus.setText(R.string.err_index_failed)
// Configure the action to act as a reload trigger. // Configure the action to act as a reload trigger.
binding.homeIndexingTry.apply { binding.homeIndexingTry.apply {
visibility = View.VISIBLE visibility = View.VISIBLE
@ -470,16 +470,14 @@ class HomeFragment :
binding.homeIndexingProgress.visibility = View.VISIBLE binding.homeIndexingProgress.visibility = View.VISIBLE
binding.homeIndexingActions.visibility = View.INVISIBLE binding.homeIndexingActions.visibility = View.INVISIBLE
binding.homeIndexingStatus.setText(R.string.lng_indexing)
when (progress) { when (progress) {
is IndexingProgress.Indeterminate -> { is IndexingProgress.Indeterminate -> {
// In a query/initialization state, show a generic loading status. // In a query/initialization state, show a generic loading status.
binding.homeIndexingStatus.text = getString(R.string.lng_indexing)
binding.homeIndexingProgress.isIndeterminate = true binding.homeIndexingProgress.isIndeterminate = true
} }
is IndexingProgress.Songs -> { is IndexingProgress.Songs -> {
// Actively loading songs, show the current progress. // Actively loading songs, show the current progress.
binding.homeIndexingStatus.text =
getString(R.string.fmt_indexing, progress.current, progress.total)
binding.homeIndexingProgress.apply { binding.homeIndexingProgress.apply {
isIndeterminate = false isIndeterminate = false
max = progress.total max = progress.total

4
app/src/main/java/org/oxycblt/auxio/music/metadata/TagExtractor.kt
View file

@ -89,8 +89,8 @@ private const val MESSAGE_RELEASE_ALL = 3
private const val CHECK_INTERVAL_MS = 100 private const val CHECK_INTERVAL_MS = 100
/** /**
* Patched version of Media3's MetadataRetriever that extracts metadata from several tracks at * Patched version of Media3's MetadataRetriever that extracts metadata from several tracks at once
* once on one thread. This is generally more efficient than stacking several threads at once. * on one thread. This is generally more efficient than stacking several threads at once.
* *
* @author Media3 Team, Alexander Capehart (OxygenCobalt) * @author Media3 Team, Alexander Capehart (OxygenCobalt)
*/ */

1
app/src/main/res/layout/fragment_home.xml
View file

@ -90,6 +90,7 @@
android:layout_height="wrap_content" android:layout_height="wrap_content"
android:layout_margin="@dimen/spacing_medium" android:layout_margin="@dimen/spacing_medium"
android:gravity="center" android:gravity="center"
android:text="@string/lng_indexing"
android:textAppearance="@style/TextAppearance.Auxio.BodyLarge" android:textAppearance="@style/TextAppearance.Auxio.BodyLarge"
app:layout_constraintBottom_toTopOf="@+id/home_indexing_actions" app:layout_constraintBottom_toTopOf="@+id/home_indexing_actions"
app:layout_constraintTop_toTopOf="parent" app:layout_constraintTop_toTopOf="parent"