From 39a990edcc37284df359ec409c32bb8fcde04cce Mon Sep 17 00:00:00 2001 From: ByteHamster Date: Thu, 24 Jun 2021 23:28:00 +0200 Subject: Speed up loading media info Loading chapters can take around 5-10 seconds, depending on the media type. During that time, the player screen shows nothing or the old media file. Instead, load the chapters afterwards. --- .../antennapod/fragment/AudioPlayerFragment.java | 28 +++++++++------- .../danoeh/antennapod/fragment/CoverFragment.java | 38 ++++++++++++++-------- .../de/danoeh/antennapod/view/ChapterSeekBar.java | 1 + 3 files changed, 42 insertions(+), 25 deletions(-) (limited to 'app/src/main/java/de/danoeh/antennapod') diff --git a/app/src/main/java/de/danoeh/antennapod/fragment/AudioPlayerFragment.java b/app/src/main/java/de/danoeh/antennapod/fragment/AudioPlayerFragment.java index 21a1d2039..a84c34b7e 100644 --- a/app/src/main/java/de/danoeh/antennapod/fragment/AudioPlayerFragment.java +++ b/app/src/main/java/de/danoeh/antennapod/fragment/AudioPlayerFragment.java @@ -268,7 +268,7 @@ public class AudioPlayerFragment extends Fragment implements } controller.setPlaybackSpeed(newSpeed); - loadMediaInfo(); + loadMediaInfo(false); }); butPlaybackSpeed.setOnLongClickListener(v -> { new VariableSpeedDialog().show(getChildFragmentManager(), null); @@ -290,14 +290,16 @@ public class AudioPlayerFragment extends Fragment implements txtvPlaybackSpeed.setVisibility(View.VISIBLE); } - private void loadMediaInfo() { + private void loadMediaInfo(boolean includingChapters) { if (disposable != null) { disposable.dispose(); } - disposable = Maybe.create(emitter -> { + disposable = Maybe.create(emitter -> { Playable media = controller.getMedia(); if (media != null) { - ChapterUtils.loadChapters(media, getContext()); + if (includingChapters) { + ChapterUtils.loadChapters(media, getContext()); + } emitter.onSuccess(media); } else { emitter.onComplete(); @@ -305,9 +307,13 @@ public class AudioPlayerFragment extends Fragment implements }) .subscribeOn(Schedulers.io()) .observeOn(AndroidSchedulers.mainThread()) - .subscribe(media -> updateUi((Playable) media), - error -> Log.e(TAG, Log.getStackTraceString(error)), - () -> updateUi(null)); + .subscribe(media -> { + updateUi(media); + if (media.getChapters() == null && !includingChapters) { + loadMediaInfo(true); + } + }, error -> Log.e(TAG, Log.getStackTraceString(error)), + () -> updateUi(null)); } private PlaybackController newPlaybackController() { @@ -350,7 +356,7 @@ public class AudioPlayerFragment extends Fragment implements @Override public void onSleepTimerUpdate() { - AudioPlayerFragment.this.loadMediaInfo(); + AudioPlayerFragment.this.loadMediaInfo(false); } @Override @@ -360,7 +366,7 @@ public class AudioPlayerFragment extends Fragment implements @Override public void loadMediaInfo() { - AudioPlayerFragment.this.loadMediaInfo(); + AudioPlayerFragment.this.loadMediaInfo(false); } @Override @@ -397,7 +403,7 @@ public class AudioPlayerFragment extends Fragment implements super.onStart(); controller = newPlaybackController(); controller.init(); - loadMediaInfo(); + loadMediaInfo(false); EventBus.getDefault().register(this); txtvRev.setText(NumberFormat.getInstance().format(UserPreferences.getRewindSecs())); txtvFF.setText(NumberFormat.getInstance().format(UserPreferences.getFastForwardSecs())); @@ -447,7 +453,7 @@ public class AudioPlayerFragment extends Fragment implements @Subscribe(threadMode = ThreadMode.MAIN) public void favoritesChanged(FavoritesEvent event) { - AudioPlayerFragment.this.loadMediaInfo(); + AudioPlayerFragment.this.loadMediaInfo(false); } @Override diff --git a/app/src/main/java/de/danoeh/antennapod/fragment/CoverFragment.java b/app/src/main/java/de/danoeh/antennapod/fragment/CoverFragment.java index 0dd97098e..fab4ff441 100644 --- a/app/src/main/java/de/danoeh/antennapod/fragment/CoverFragment.java +++ b/app/src/main/java/de/danoeh/antennapod/fragment/CoverFragment.java @@ -124,14 +124,16 @@ public class CoverFragment extends Fragment { configureForOrientation(getResources().getConfiguration()); } - private void loadMediaInfo() { + private void loadMediaInfo(boolean includingChapters) { if (disposable != null) { disposable.dispose(); } disposable = Maybe.create(emitter -> { Playable media = controller.getMedia(); if (media != null) { - ChapterUtils.loadChapters(media, getContext()); + if (includingChapters) { + ChapterUtils.loadChapters(media, getContext()); + } emitter.onSuccess(media); } else { emitter.onComplete(); @@ -141,6 +143,9 @@ public class CoverFragment extends Fragment { .subscribe(media -> { this.media = media; displayMediaInfo(media); + if (media.getChapters() == null && !includingChapters) { + loadMediaInfo(true); + } }, error -> Log.e(TAG, Log.getStackTraceString(error))); } @@ -186,17 +191,22 @@ public class CoverFragment extends Fragment { } private void updateChapterControlVisibility() { + boolean chapterControlVisible = false; if (media.getChapters() != null) { - boolean chapterControlVisible = media.getChapters().size() > 0; - int newVisibility = chapterControlVisible ? View.VISIBLE : View.GONE; - if (chapterControl.getVisibility() != newVisibility) { - chapterControl.setVisibility(newVisibility); - ObjectAnimator.ofFloat(chapterControl, - "alpha", - chapterControlVisible ? 0 : 1, - chapterControlVisible ? 1 : 0) - .start(); - } + chapterControlVisible = media.getChapters().size() > 0; + } else if (media instanceof FeedMedia) { + FeedMedia fm = ((FeedMedia) media); + // If an item has chapters but they are not loaded yet, still display the button. + chapterControlVisible = fm.getItem() != null && fm.getItem().hasChapters(); + } + int newVisibility = chapterControlVisible ? View.VISIBLE : View.GONE; + if (chapterControl.getVisibility() != newVisibility) { + chapterControl.setVisibility(newVisibility); + ObjectAnimator.ofFloat(chapterControl, + "alpha", + chapterControlVisible ? 0 : 1, + chapterControlVisible ? 1 : 0) + .start(); } } @@ -262,11 +272,11 @@ public class CoverFragment extends Fragment { controller = new PlaybackController(getActivity()) { @Override public void loadMediaInfo() { - CoverFragment.this.loadMediaInfo(); + CoverFragment.this.loadMediaInfo(false); } }; controller.init(); - loadMediaInfo(); + loadMediaInfo(false); EventBus.getDefault().register(this); } diff --git a/app/src/main/java/de/danoeh/antennapod/view/ChapterSeekBar.java b/app/src/main/java/de/danoeh/antennapod/view/ChapterSeekBar.java index ba7acb847..7c6b64e2c 100644 --- a/app/src/main/java/de/danoeh/antennapod/view/ChapterSeekBar.java +++ b/app/src/main/java/de/danoeh/antennapod/view/ChapterSeekBar.java @@ -62,6 +62,7 @@ public class ChapterSeekBar extends androidx.appcompat.widget.AppCompatSeekBar { } else { this.dividerPos = null; } + invalidate(); } public void highlightCurrentChapter() { -- cgit v1.2.3 From 951aa3dce6bbc6cace9e9b6bcf0862749576233a Mon Sep 17 00:00:00 2001 From: vbh <56578479+vbh@users.noreply.github.com> Date: Mon, 28 Jun 2021 20:02:03 +0000 Subject: Allow to choose from existing tags/folders in folder settings dialog (#5245) --- .../antennapod/dialog/TagSettingsDialog.java | 45 ++++++++++++++++++++++ 1 file changed, 45 insertions(+) (limited to 'app/src/main/java/de/danoeh/antennapod') diff --git a/app/src/main/java/de/danoeh/antennapod/dialog/TagSettingsDialog.java b/app/src/main/java/de/danoeh/antennapod/dialog/TagSettingsDialog.java index de9f4d504..8ef01590f 100644 --- a/app/src/main/java/de/danoeh/antennapod/dialog/TagSettingsDialog.java +++ b/app/src/main/java/de/danoeh/antennapod/dialog/TagSettingsDialog.java @@ -3,7 +3,11 @@ package de.danoeh.antennapod.dialog; import android.app.Dialog; import android.os.Bundle; import android.text.TextUtils; +import android.util.Log; +import android.view.MotionEvent; +import android.view.View; import android.view.ViewGroup; +import android.widget.ArrayAdapter; import androidx.annotation.NonNull; import androidx.annotation.Nullable; import androidx.appcompat.app.AlertDialog; @@ -12,10 +16,15 @@ import androidx.recyclerview.widget.GridLayoutManager; import androidx.recyclerview.widget.RecyclerView; import com.google.android.material.chip.Chip; import de.danoeh.antennapod.R; +import de.danoeh.antennapod.core.storage.DBReader; +import de.danoeh.antennapod.core.storage.NavDrawerData; import de.danoeh.antennapod.model.feed.FeedPreferences; import de.danoeh.antennapod.core.storage.DBWriter; import de.danoeh.antennapod.databinding.EditTagsDialogBinding; import de.danoeh.antennapod.view.ItemOffsetDecoration; +import io.reactivex.Observable; +import io.reactivex.android.schedulers.AndroidSchedulers; +import io.reactivex.schedulers.Schedulers; import java.util.ArrayList; import java.util.List; @@ -53,6 +62,17 @@ public class TagSettingsDialog extends DialogFragment { viewBinding.newTagButton.setOnClickListener(v -> addTag(viewBinding.newTagEditText.getText().toString().trim())); + loadTags(); + viewBinding.newTagEditText.setThreshold(1); + viewBinding.newTagEditText.setOnTouchListener(new View.OnTouchListener() { + @Override + public boolean onTouch(View v, MotionEvent event) { + viewBinding.newTagEditText.showDropDown(); + viewBinding.newTagEditText.requestFocus(); + return false; + } + }); + AlertDialog.Builder dialog = new AlertDialog.Builder(getContext()); dialog.setView(viewBinding.getRoot()); dialog.setTitle(R.string.feed_folders_label); @@ -69,6 +89,31 @@ public class TagSettingsDialog extends DialogFragment { return dialog.create(); } + private void loadTags() { + Observable.fromCallable( + () -> { + NavDrawerData data = DBReader.getNavDrawerData(); + List items = data.items; + List folders = new ArrayList(); + for (NavDrawerData.DrawerItem item : items) { + if (item.type == NavDrawerData.DrawerItem.Type.FOLDER) { + folders.add(item.getTitle()); + } + } + return folders; + }) + .subscribeOn(Schedulers.io()) + .observeOn(AndroidSchedulers.mainThread()) + .subscribe( + result -> { + ArrayAdapter acAdapter = new ArrayAdapter(getContext(), + R.layout.single_tag_text_view, result); + viewBinding.newTagEditText.setAdapter(acAdapter); + }, error -> { + Log.e(TAG, Log.getStackTraceString(error)); + }); + } + private void addTag(String name) { if (TextUtils.isEmpty(name) || displayedTags.contains(name)) { return; -- cgit v1.2.3 From 3cab62348a236f0b836f684f7c2aaa32a765cdc5 Mon Sep 17 00:00:00 2001 From: ByteHamster Date: Sat, 24 Jul 2021 08:38:25 +0200 Subject: Removed highlighting of episode title --- app/src/main/java/de/danoeh/antennapod/fragment/CoverFragment.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'app/src/main/java/de/danoeh/antennapod') diff --git a/app/src/main/java/de/danoeh/antennapod/fragment/CoverFragment.java b/app/src/main/java/de/danoeh/antennapod/fragment/CoverFragment.java index 0dd97098e..9c9eee2aa 100644 --- a/app/src/main/java/de/danoeh/antennapod/fragment/CoverFragment.java +++ b/app/src/main/java/de/danoeh/antennapod/fragment/CoverFragment.java @@ -161,7 +161,9 @@ public class CoverFragment extends Fragment { int animUnit = 1500; if (lines > txtvEpisodeTitle.getMaxLines()) { ObjectAnimator verticalMarquee = ObjectAnimator.ofInt( - txtvEpisodeTitle, "scrollY", 0, txtvEpisodeTitle.getHeight()) + txtvEpisodeTitle, "scrollY", 0, (lines - txtvEpisodeTitle.getMaxLines()) * ( + (txtvEpisodeTitle.getHeight() - txtvEpisodeTitle.getPaddingTop() + - txtvEpisodeTitle.getPaddingBottom()) / txtvEpisodeTitle.getMaxLines())) .setDuration(lines * animUnit); ObjectAnimator fadeOut = ObjectAnimator.ofFloat( txtvEpisodeTitle, "alpha", 0); -- cgit v1.2.3 From fb455b982e42eeac52735050ebe4ebd80c6129df Mon Sep 17 00:00:00 2001 From: ByteHamster Date: Sun, 25 Jul 2021 08:45:25 +0200 Subject: Fix exporting full logs --- .../main/java/de/danoeh/antennapod/activity/BugReportActivity.java | 4 ---- 1 file changed, 4 deletions(-) (limited to 'app/src/main/java/de/danoeh/antennapod') diff --git a/app/src/main/java/de/danoeh/antennapod/activity/BugReportActivity.java b/app/src/main/java/de/danoeh/antennapod/activity/BugReportActivity.java index 50794ba5b..aa59e4e96 100644 --- a/app/src/main/java/de/danoeh/antennapod/activity/BugReportActivity.java +++ b/app/src/main/java/de/danoeh/antennapod/activity/BugReportActivity.java @@ -99,10 +99,6 @@ public class BugReportActivity extends AppCompatActivity { private void exportLog() { try { File filename = new File(UserPreferences.getDataFolder(null), "full-logs.txt"); - boolean success = filename.createNewFile(); - if (!success) { - throw new IOException("Unable to create output file"); - } String cmd = "logcat -d -f " + filename.getAbsolutePath(); Runtime.getRuntime().exec(cmd); //share file -- cgit v1.2.3