From 4cc433f7ef73cd5b840012c7ba32ea6fb324a071 Mon Sep 17 00:00:00 2001 From: OxygenCobalt Date: Sun, 8 Aug 2021 14:05:44 -0600 Subject: [PATCH] style: fix style issues Fix some style issues that came from the colorAccent reworks. --- README.md | 2 +- app/build.gradle | 6 +++--- .../java/org/oxycblt/auxio/MainFragment.kt | 1 - .../auxio/playback/PlaybackFragment.kt | 2 +- .../auxio/playback/PlaybackViewModel.kt | 4 ++-- .../auxio/settings/accent/AccentAdapter.kt | 2 +- .../auxio/settings/accent/AccentDialog.kt | 3 ++- .../settings/blacklist/BlacklistDialog.kt | 2 ++ .../auxio/settings/ui/LifecycleDialog.kt | 2 +- .../org/oxycblt/auxio/ui/InterfaceUtils.kt | 2 +- .../oxycblt/auxio/widgets/WidgetProvider.kt | 5 ++--- app/src/main/res/drawable/ic_pause_large.xml | 2 +- app/src/main/res/drawable/ic_play_large.xml | 2 +- .../main/res/drawable/ic_queue_inactive.xml | 2 +- .../main/res/drawable/ui_header_dividers.xml | 2 +- app/src/main/res/layout/fragment_about.xml | 2 +- app/src/main/res/values-de/strings.xml | 2 +- app/src/main/res/values-es/strings.xml | 2 +- .../main/res/values-night-v27/styles_core.xml | 2 +- app/src/main/res/values-night/colors.xml | 12 ++++++------ app/src/main/res/values-nl/strings.xml | 2 +- app/src/main/res/values-v27/styles_core.xml | 2 +- app/src/main/res/values-v31/styles_core.xml | 1 + app/src/main/res/values/colors.xml | 14 +++++++------- app/src/main/res/values/strings.xml | 2 +- app/src/main/res/values/styles_android.xml | 7 ++++--- app/src/main/res/values/styles_core.xml | 19 +++++++++++-------- app/src/main/res/values/styles_ui.xml | 7 ++++--- app/src/main/res/values/themes.xml | 2 +- app/src/main/res/xml/prefs_main.xml | 2 +- 30 files changed, 62 insertions(+), 55 deletions(-) diff --git a/README.md b/README.md index 5063c8ef3..a958860ac 100644 --- a/README.md +++ b/README.md @@ -21,7 +21,7 @@ Auxio is a local music player with a fast, reliable UI/UX without the many usele I primarily built Auxio for myself, but you can use it too, I guess. -#### Note: Auxio is in a point that I am largely satisfied with. The app is still maintained, but feature additions will be slow. +**Note: Auxio is in a point that I am largely satisfied with. The app is still maintained, but feature additions will be slow.** ## Screenshots diff --git a/app/build.gradle b/app/build.gradle index a1e4844e8..75bd83da6 100644 --- a/app/build.gradle +++ b/app/build.gradle @@ -13,7 +13,7 @@ android { versionCode 7 minSdkVersion 21 - targetSdkVersion 30 + targetSdkVersion 30 // Note: Do not increment until platform stability buildFeatures { dataBinding true @@ -94,7 +94,7 @@ dependencies { implementation "com.google.android.exoplayer:exoplayer-core:2.14.2" // Image loading - implementation 'io.coil-kt:coil:1.3.1' + implementation 'io.coil-kt:coil:1.3.2' // Material implementation "com.google.android.material:material:1.4.0" @@ -102,7 +102,7 @@ dependencies { // --- DEBUG --- // Lint - ktlint 'com.pinterest:ktlint:0.42.0' + ktlint 'com.pinterest:ktlint:0.42.1' // Memory Leak checking debugImplementation 'com.squareup.leakcanary:leakcanary-android:2.7' diff --git a/app/src/main/java/org/oxycblt/auxio/MainFragment.kt b/app/src/main/java/org/oxycblt/auxio/MainFragment.kt index d48ea26cd..b73bc9c99 100644 --- a/app/src/main/java/org/oxycblt/auxio/MainFragment.kt +++ b/app/src/main/java/org/oxycblt/auxio/MainFragment.kt @@ -134,7 +134,6 @@ class MainFragment : Fragment() { */ private fun navigateWithItem(navController: NavController, item: MenuItem): Boolean { if (navController.currentDestination!!.id != item.itemId) { - // Create custom NavOptions myself so that animations work val options = NavOptions.Builder() .setLaunchSingleTop(true) .setEnterAnim(R.animator.nav_default_enter_anim) diff --git a/app/src/main/java/org/oxycblt/auxio/playback/PlaybackFragment.kt b/app/src/main/java/org/oxycblt/auxio/playback/PlaybackFragment.kt index a8a6d120b..b863ab89e 100644 --- a/app/src/main/java/org/oxycblt/auxio/playback/PlaybackFragment.kt +++ b/app/src/main/java/org/oxycblt/auxio/playback/PlaybackFragment.kt @@ -56,7 +56,7 @@ class PlaybackFragment : Fragment(), SeekBar.OnSeekBarChangeListener { ): View { val normalTextColor = binding.playbackDurationCurrent.currentTextColor val accentColor = Accent.get().getStateList(requireContext()) - val controlColor = R.color.control_color.toStateList(requireContext()) + val controlColor = R.color.control.toStateList(requireContext()) // Can't set the tint of a MenuItem below Android 8, so use icons instead. val iconQueueActive = R.drawable.ic_queue.toDrawable(requireContext()) diff --git a/app/src/main/java/org/oxycblt/auxio/playback/PlaybackViewModel.kt b/app/src/main/java/org/oxycblt/auxio/playback/PlaybackViewModel.kt index 508620771..bf1e2ef57 100644 --- a/app/src/main/java/org/oxycblt/auxio/playback/PlaybackViewModel.kt +++ b/app/src/main/java/org/oxycblt/auxio/playback/PlaybackViewModel.kt @@ -436,14 +436,14 @@ class PlaybackViewModel : ViewModel(), PlaybackStateManager.Callback { } /** - * Enable animation on CompactPlaybackFragment + * Enable animation on the play/pause button, usually done after the first LiveData fire */ fun enableAnimation() { mCanAnimate = true } /** - * Disable animation on CompactPlaybackFragment + * Disable animation on play/pause button, usually done after when a UI is created */ fun disableAnimation() { mCanAnimate = false diff --git a/app/src/main/java/org/oxycblt/auxio/settings/accent/AccentAdapter.kt b/app/src/main/java/org/oxycblt/auxio/settings/accent/AccentAdapter.kt index 10553df84..8153c4ac0 100644 --- a/app/src/main/java/org/oxycblt/auxio/settings/accent/AccentAdapter.kt +++ b/app/src/main/java/org/oxycblt/auxio/settings/accent/AccentAdapter.kt @@ -84,7 +84,7 @@ class AccentAdapter( selectedViewHolder?.setSelected(false) selectedViewHolder = this - R.color.surface_color.toStateList(context) + R.color.surface.toStateList(context) } else { android.R.color.transparent.toStateList(context) } diff --git a/app/src/main/java/org/oxycblt/auxio/settings/accent/AccentDialog.kt b/app/src/main/java/org/oxycblt/auxio/settings/accent/AccentDialog.kt index deb450940..20b25a314 100644 --- a/app/src/main/java/org/oxycblt/auxio/settings/accent/AccentDialog.kt +++ b/app/src/main/java/org/oxycblt/auxio/settings/accent/AccentDialog.kt @@ -23,6 +23,7 @@ import android.view.LayoutInflater import android.view.View import android.view.ViewGroup import androidx.appcompat.app.AlertDialog +import org.oxycblt.auxio.BuildConfig import org.oxycblt.auxio.R import org.oxycblt.auxio.databinding.DialogAccentBinding import org.oxycblt.auxio.logD @@ -102,6 +103,6 @@ class AccentDialog : LifecycleDialog() { companion object { const val TAG = "TAG_ACCENT_DIALOG" - const val KEY_PENDING_ACCENT = "AXKEY_PEND_ACCENT" + const val KEY_PENDING_ACCENT = BuildConfig.APPLICATION_ID + ".key.PENDING_ACCENT" } } diff --git a/app/src/main/java/org/oxycblt/auxio/settings/blacklist/BlacklistDialog.kt b/app/src/main/java/org/oxycblt/auxio/settings/blacklist/BlacklistDialog.kt index 2fa31c649..97810ac2b 100644 --- a/app/src/main/java/org/oxycblt/auxio/settings/blacklist/BlacklistDialog.kt +++ b/app/src/main/java/org/oxycblt/auxio/settings/blacklist/BlacklistDialog.kt @@ -137,6 +137,8 @@ class BlacklistDialog : LifecycleDialog() { val typeAndPath = DocumentsContract.getTreeDocumentId(docUri).split(":") // Only the main drive is supported, since thats all we can get from MediaColumns.DATA + // Unless I change the system to use the drive/directory system, but thats limited to + // Android 10 if (typeAndPath[0] == "primary") { return getRootPath() + "/" + typeAndPath.last() } diff --git a/app/src/main/java/org/oxycblt/auxio/settings/ui/LifecycleDialog.kt b/app/src/main/java/org/oxycblt/auxio/settings/ui/LifecycleDialog.kt index b0a092c02..5dc7e45c5 100644 --- a/app/src/main/java/org/oxycblt/auxio/settings/ui/LifecycleDialog.kt +++ b/app/src/main/java/org/oxycblt/auxio/settings/ui/LifecycleDialog.kt @@ -37,7 +37,7 @@ abstract class LifecycleDialog : AppCompatDialogFragment() { override fun onCreateDialog(savedInstanceState: Bundle?): Dialog { val builder = MaterialAlertDialogBuilder(requireActivity(), theme) - // Setting the background in XML will also apply it to the tooltip for some inane reason + // Setting the background in XML will also apply it to the tooltip for some insane reason // so we have to do it programmatically instead. builder.background = R.attr.colorSurface.resolveAttr(requireContext()).toDrawable() diff --git a/app/src/main/java/org/oxycblt/auxio/ui/InterfaceUtils.kt b/app/src/main/java/org/oxycblt/auxio/ui/InterfaceUtils.kt index 397497ca5..f4723faa4 100644 --- a/app/src/main/java/org/oxycblt/auxio/ui/InterfaceUtils.kt +++ b/app/src/main/java/org/oxycblt/auxio/ui/InterfaceUtils.kt @@ -58,7 +58,7 @@ const val INTENT_REQUEST_CODE = 0xA0A0 */ fun ImageButton.disable() { if (isEnabled) { - imageTintList = R.color.inactive_color.toStateList(context) + imageTintList = R.color.inactive.toStateList(context) isEnabled = false } } diff --git a/app/src/main/java/org/oxycblt/auxio/widgets/WidgetProvider.kt b/app/src/main/java/org/oxycblt/auxio/widgets/WidgetProvider.kt index b168e36ca..7c4ac2de7 100644 --- a/app/src/main/java/org/oxycblt/auxio/widgets/WidgetProvider.kt +++ b/app/src/main/java/org/oxycblt/auxio/widgets/WidgetProvider.kt @@ -124,9 +124,8 @@ class WidgetProvider : AppWidgetProvider() { updateAppWidget(name, RemoteViews(views)) } else { // Otherwise, we try our best to backport the responsive behavior to older versions. - // This is mostly a guess based on RemoteView's documentation, and it has some - // problems [most notably UI jittering when resizing]. Its better than just using - // one layout though. It may be improved once Android 12's source is released. + // This is mostly a guess based on RemoteView's documentation. It may be improved when + // Android 12's source is released. // Each widget has independent dimensions, so we iterate through them all // and do this for each. diff --git a/app/src/main/res/drawable/ic_pause_large.xml b/app/src/main/res/drawable/ic_pause_large.xml index d6671227c..d52e62c9b 100644 --- a/app/src/main/res/drawable/ic_pause_large.xml +++ b/app/src/main/res/drawable/ic_pause_large.xml @@ -2,7 +2,7 @@ + android:color="@color/divider" /> diff --git a/app/src/main/res/layout/fragment_about.xml b/app/src/main/res/layout/fragment_about.xml index 00cbf45e5..737a4ef15 100644 --- a/app/src/main/res/layout/fragment_about.xml +++ b/app/src/main/res/layout/fragment_about.xml @@ -27,7 +27,7 @@ app:cardBackgroundColor="?attr/colorSurface" app:cardCornerRadius="0dp" app:cardElevation="0dp" - app:strokeColor="@color/divider_color" + app:strokeColor="@color/divider" app:strokeWidth="1dp"> Musikbibliothekitems Benachrichtigung farblich anpassen Albumcover am Benachrichtigung zeigen - Albumcover anzeigen + Albumcover anzeigen Ausschalten zu Speicherverbrauch sparen MediaStore Cover ignorieren Verbesst die Albumcover Qualität, jedoch länger Ladezeit und höher Speicherverbrauch beeinträchtigt diff --git a/app/src/main/res/values-es/strings.xml b/app/src/main/res/values-es/strings.xml index e6cb62e1e..187866eff 100644 --- a/app/src/main/res/values-es/strings.xml +++ b/app/src/main/res/values-es/strings.xml @@ -68,7 +68,7 @@ Ítem de biblioteca Ajustar color de notificación Mostrar imagen de álbum en la notificación - Mostrar carátula de álbum + Mostrar carátula de álbum Desactivar para ahorrar uso de memoria Ignorar carátulas de MediaStore Mejora la calidad de las carátulas de álbum, pero resulta en tiempos de carga lentos y un mayor uso de memoria diff --git a/app/src/main/res/values-night-v27/styles_core.xml b/app/src/main/res/values-night-v27/styles_core.xml index 04dda1dce..4dc7dc273 100644 --- a/app/src/main/res/values-night-v27/styles_core.xml +++ b/app/src/main/res/values-night-v27/styles_core.xml @@ -3,7 +3,7 @@ diff --git a/app/src/main/res/values-night/colors.xml b/app/src/main/res/values-night/colors.xml index 4b02d8367..4823af197 100644 --- a/app/src/main/res/values-night/colors.xml +++ b/app/src/main/res/values-night/colors.xml @@ -1,11 +1,11 @@ - @color/surface_night - #323232 - #484848 - #404040 - #ffffff - #01151515 + @color/surface_night + #323232 + #484848 + #404040 + #ffffff + #01151515 diff --git a/app/src/main/res/values-v31/styles_core.xml b/app/src/main/res/values-v31/styles_core.xml index 959a314df..e874a528b 100644 --- a/app/src/main/res/values-v31/styles_core.xml +++ b/app/src/main/res/values-v31/styles_core.xml @@ -1,5 +1,6 @@ + diff --git a/app/src/main/res/values/colors.xml b/app/src/main/res/values/colors.xml index 9efaef436..474403059 100644 --- a/app/src/main/res/values/colors.xml +++ b/app/src/main/res/values/colors.xml @@ -1,15 +1,15 @@ #fffafafa - #ff151515 + #ff121212 #ff000000 - @color/surface_day - #cbcbcb - #202020 - #cbcbcb - #c4c4c4 - #01fafafa + @color/surface_day + #cbcbcb + #202020 + #cbcbcb + #c4c4c4 + #01fafafa @@ -30,8 +30,8 @@ @@ -53,6 +53,7 @@ diff --git a/app/src/main/res/values/styles_core.xml b/app/src/main/res/values/styles_core.xml index 7de162326..78a41296f 100644 --- a/app/src/main/res/values/styles_core.xml +++ b/app/src/main/res/values/styles_core.xml @@ -4,19 +4,22 @@ - \ No newline at end of file diff --git a/app/src/main/res/values/themes.xml b/app/src/main/res/values/themes.xml index 2fd36e841..138558275 100644 --- a/app/src/main/res/values/themes.xml +++ b/app/src/main/res/values/themes.xml @@ -76,6 +76,6 @@ \ No newline at end of file diff --git a/app/src/main/res/xml/prefs_main.xml b/app/src/main/res/xml/prefs_main.xml index ddfc42332..edbfb46e1 100644 --- a/app/src/main/res/xml/prefs_main.xml +++ b/app/src/main/res/xml/prefs_main.xml @@ -48,7 +48,7 @@ app:iconSpaceReserved="false" app:key="KEY_SHOW_COVERS" app:summary="@string/set_show_covers_desc" - app:title="@string/setting_show_covers" /> + app:title="@string/set_show_covers" />