diff --git a/app/src/main/java/code/name/monkey/retromusic/activities/AboutActivity.kt b/app/src/main/java/code/name/monkey/retromusic/activities/AboutActivity.kt index d3e3f1ee6..8f4341722 100644 --- a/app/src/main/java/code/name/monkey/retromusic/activities/AboutActivity.kt +++ b/app/src/main/java/code/name/monkey/retromusic/activities/AboutActivity.kt @@ -9,6 +9,7 @@ import android.view.View import androidx.core.app.ShareCompat import androidx.recyclerview.widget.DefaultItemAnimator import androidx.recyclerview.widget.LinearLayoutManager +import code.name.monkey.appthemehelper.util.ATHUtil import code.name.monkey.appthemehelper.util.ToolbarContentTintHelper import code.name.monkey.retromusic.Constants.APP_INSTAGRAM_LINK import code.name.monkey.retromusic.Constants.APP_TELEGRAM_LINK @@ -68,8 +69,13 @@ class AboutActivity : AbsBaseActivity(), View.OnClickListener { loadContributors() setSupportActionBar(toolbar) - ToolbarContentTintHelper.colorBackButton(toolbar ) - version.setSummary ( getAppVersion()) + toolbar.apply { + setTitleTextColor(ATHUtil.resolveColor(this@AboutActivity, R.attr.colorOnPrimary)) + setBackgroundColor(ATHUtil.resolveColor(this@AboutActivity, R.attr.colorPrimary)) + setNavigationOnClickListener { onBackPressed() } + ToolbarContentTintHelper.colorBackButton(toolbar) + } + version.setSummary(getAppVersion()) setUpView() } diff --git a/app/src/main/java/code/name/monkey/retromusic/activities/LicenseActivity.java b/app/src/main/java/code/name/monkey/retromusic/activities/LicenseActivity.java index 6b9470831..7c90f1764 100644 --- a/app/src/main/java/code/name/monkey/retromusic/activities/LicenseActivity.java +++ b/app/src/main/java/code/name/monkey/retromusic/activities/LicenseActivity.java @@ -30,6 +30,7 @@ import java.io.InputStreamReader; import java.nio.charset.StandardCharsets; import code.name.monkey.appthemehelper.ThemeStore; +import code.name.monkey.appthemehelper.util.ATHUtil; import code.name.monkey.appthemehelper.util.ColorUtil; import code.name.monkey.appthemehelper.util.ToolbarContentTintHelper; import code.name.monkey.retromusic.R; @@ -61,11 +62,11 @@ public class LicenseActivity extends AbsBaseActivity { setContentView(R.layout.activity_license); setStatusbarColorAuto(); setNavigationBarColorPrimary(); - setTaskDescriptionColorAuto(); setLightNavigationBar(true); Toolbar toolbar = findViewById(R.id.toolbar); setSupportActionBar(toolbar); - ToolbarContentTintHelper.colorBackButton(toolbar ); + ToolbarContentTintHelper.colorBackButton(toolbar); + toolbar.setBackgroundColor(INSTANCE.resolveColor(this, R.attr.colorPrimary)); WebView webView = findViewById(R.id.license); try { StringBuilder buf = new StringBuilder(); diff --git a/app/src/main/java/code/name/monkey/retromusic/activities/base/AbsThemeActivity.kt b/app/src/main/java/code/name/monkey/retromusic/activities/base/AbsThemeActivity.kt index 7d802ec13..603f13f20 100644 --- a/app/src/main/java/code/name/monkey/retromusic/activities/base/AbsThemeActivity.kt +++ b/app/src/main/java/code/name/monkey/retromusic/activities/base/AbsThemeActivity.kt @@ -29,7 +29,6 @@ abstract class AbsThemeActivity : ATHToolbarActivity(), Runnable { setTheme(ThemeManager.getThemeResValue(this)) hideStatusBar() super.onCreate(savedInstanceState) - //MaterialDialogsUtil.updateMaterialDialogsThemeSingleton(this) changeBackgroundShape() setImmersiveFullscreen() diff --git a/app/src/main/java/code/name/monkey/retromusic/adapter/song/ShuffleButtonSongAdapter.kt b/app/src/main/java/code/name/monkey/retromusic/adapter/song/ShuffleButtonSongAdapter.kt index ec066c10b..fe1136787 100644 --- a/app/src/main/java/code/name/monkey/retromusic/adapter/song/ShuffleButtonSongAdapter.kt +++ b/app/src/main/java/code/name/monkey/retromusic/adapter/song/ShuffleButtonSongAdapter.kt @@ -3,8 +3,6 @@ package code.name.monkey.retromusic.adapter.song import android.view.View import androidx.annotation.LayoutRes import androidx.appcompat.app.AppCompatActivity -import code.name.monkey.appthemehelper.util.ATHUtil -import code.name.monkey.appthemehelper.util.MaterialUtil import code.name.monkey.retromusic.R import code.name.monkey.retromusic.helper.MusicPlayerRemote import code.name.monkey.retromusic.interfaces.CabHolder @@ -27,13 +25,11 @@ class ShuffleButtonSongAdapter(activity: AppCompatActivity, if (holder.itemViewType == OFFSET_ITEM) { val viewHolder = holder as ViewHolder viewHolder.playAction?.let { - MaterialUtil.setTint(it, color = ATHUtil.resolveColor(activity, R.attr.cardBackgroundColor)) it.setOnClickListener { MusicPlayerRemote.openQueue(dataSet, 0, true) } } viewHolder.shuffleAction?.let { - MaterialUtil.setTint(button = it, color = ATHUtil.resolveColor(activity, R.attr.cardBackgroundColor)) it.setOnClickListener { MusicPlayerRemote.openAndShuffleQueue(dataSet, true) } diff --git a/app/src/main/java/code/name/monkey/retromusic/fragments/base/AbsLibraryPagerRecyclerViewCustomGridSizeFragment.kt b/app/src/main/java/code/name/monkey/retromusic/fragments/base/AbsLibraryPagerRecyclerViewCustomGridSizeFragment.kt index 703c99742..09c8361bc 100644 --- a/app/src/main/java/code/name/monkey/retromusic/fragments/base/AbsLibraryPagerRecyclerViewCustomGridSizeFragment.kt +++ b/app/src/main/java/code/name/monkey/retromusic/fragments/base/AbsLibraryPagerRecyclerViewCustomGridSizeFragment.kt @@ -31,7 +31,6 @@ abstract class AbsLibraryPagerRecyclerViewCustomGridSizeFragment maxGridSizeForList) { R.layout.item_grid } else R.layout.item_list @@ -124,13 +123,11 @@ abstract class AbsLibraryPagerRecyclerViewCustomGridSizeFragment? = null - private var lyricsColor: Lyrics? = null - override fun onShow() { playbackControlsFragment!!.show() } @@ -102,7 +89,6 @@ class ColorFragment : AbsPlayerFragment() { super.onViewCreated(view, savedInstanceState) setUpSubFragments() setUpPlayerToolbar() - setupViews() } private fun setUpSubFragments() { @@ -113,7 +99,7 @@ class ColorFragment : AbsPlayerFragment() { private fun setUpPlayerToolbar() { playerToolbar.apply { inflateMenu(R.menu.menu_player) - setNavigationOnClickListener { activity!!.onBackPressed() } + setNavigationOnClickListener { requireActivity().onBackPressed() } setOnMenuItemClickListener(this@ColorFragment) ToolbarContentTintHelper.colorizeToolbar(this, ATHUtil.resolveColor(context, R.attr.iconColor), activity) } @@ -122,18 +108,14 @@ class ColorFragment : AbsPlayerFragment() { override fun onPlayingMetaChanged() { super.onPlayingMetaChanged() updateSong() - updateLyricsLocal() } override fun onServiceConnected() { super.onServiceConnected() updateSong() - updateLyricsLocal() } private fun updateSong() { - - SongGlideRequest.Builder.from(Glide.with(requireActivity()), MusicPlayerRemote.currentSong) .checkIgnoreMediaStore(requireContext()) .generatePalette(requireContext()) @@ -148,7 +130,6 @@ class ColorFragment : AbsPlayerFragment() { resource?.let { val background = resource.palette.getColor() - val palette = resource.palette val swatch = RetroColorUtil.getSwatch(palette) @@ -175,85 +156,12 @@ class ColorFragment : AbsPlayerFragment() { private fun setColors(backgroundColor: Int, textColor: Int) { playbackControlsFragment!!.setDark(textColor, backgroundColor) - colorGradientBackground?.setBackgroundColor(backgroundColor) - ToolbarContentTintHelper.colorizeToolbar(playerToolbar, textColor, activity) - lastColor = textColor - this.backgroundColor = backgroundColor - - if (playerActivity != null) { - playerActivity!!.setLightNavigationBar(ColorUtil.isColorLight(backgroundColor)) - } - callbacks!!.onPaletteColorChanged() - - } - - private fun colorize(i: Int) { - if (valueAnimator != null) { - valueAnimator!!.cancel() - } - - valueAnimator = ValueAnimator.ofObject(ArgbEvaluator(), paletteColor, i) - valueAnimator!!.addUpdateListener { animation -> - colorGradientBackground?.setBackgroundColor(animation.animatedValue as Int) - } - valueAnimator!!.setDuration(ViewUtil.RETRO_MUSIC_ANIM_TIME.toLong()).start() - } - - @SuppressLint("StaticFieldLeak") - private fun updateLyricsLocal() { - if (updateLyricsAsyncTask != null) { - updateLyricsAsyncTask!!.cancel(false) - } - val song = MusicPlayerRemote.currentSong - updateLyricsAsyncTask = object : AsyncTask() { - override fun onPreExecute() { - super.onPreExecute() - lyricsColor = null - playerToolbar.menu.removeItem(R.id.action_show_lyrics) - } - - override fun doInBackground(vararg params: Void?): Lyrics? { - val data = MusicUtil.getLyrics(song) - return if (TextUtils.isEmpty(data)) { - null - } else Lyrics.parse(song, data!!) - } - - override fun onPostExecute(l: Lyrics?) { - lyricsColor = l - if (lyricsColor == null) { - lyricsView?.setText(R.string.no_lyrics_found) - } else { - lyricsView?.text = lyricsColor!!.text - } - } - - override fun onCancelled(s: Lyrics?) { - onPostExecute(null) - } - }.execute() - } - - private fun setupViews() { - lyricsView.setOnClickListener { - if (lyricsContainer!!.visibility == View.GONE) { - lyricsContainer!!.visibility = View.VISIBLE - } else { - lyricsContainer!!.visibility = View.GONE - } - } - playerImage.setOnClickListener { - if (lyricsContainer!!.visibility == View.GONE) { - lyricsContainer!!.visibility = View.VISIBLE - } else { - lyricsContainer!!.visibility = View.GONE - } - } - expand.setOnClickListener { startActivity(Intent(context, LyricsActivity::class.java)) } + playerActivity?.setLightNavigationBar(ColorUtil.isColorLight(backgroundColor)) + callbacks?.onPaletteColorChanged() } companion object { diff --git a/app/src/main/res/layout-land/fragment_color_player.xml b/app/src/main/res/layout-land/fragment_color_player.xml index f477b3e30..ba453b825 100644 --- a/app/src/main/res/layout-land/fragment_color_player.xml +++ b/app/src/main/res/layout-land/fragment_color_player.xml @@ -36,7 +36,7 @@ android:layout_width="match_parent" android:layout_height="match_parent"> - - - - - - - - - - - + diff --git a/app/src/main/res/layout/activity_about.xml b/app/src/main/res/layout/activity_about.xml index b4045ac2d..71f1cca3c 100644 --- a/app/src/main/res/layout/activity_about.xml +++ b/app/src/main/res/layout/activity_about.xml @@ -15,9 +15,9 @@ android:id="@+id/toolbar" style="@style/Toolbar" app:layout_collapseMode="pin" - app:titleTextAppearance="@style/ToolbarTextAppearanceNormal" app:navigationIcon="@drawable/ic_keyboard_backspace_black_24dp" - app:title="@string/action_about" /> + app:title="@string/action_about" + app:titleTextAppearance="@style/ToolbarTextAppearanceNormal" /> diff --git a/app/src/main/res/layout/activity_album_content.xml b/app/src/main/res/layout/activity_album_content.xml index 11bada17f..e89994d51 100644 --- a/app/src/main/res/layout/activity_album_content.xml +++ b/app/src/main/res/layout/activity_album_content.xml @@ -14,14 +14,8 @@ android:layout_marginTop="8dp" android:layout_marginEnd="@dimen/button_margin_horizontal" android:layout_marginBottom="8dp" - android:paddingTop="@dimen/button_padding_vertical" - android:paddingBottom="@dimen/button_padding_vertical" android:text="@string/action_play_all" - android:textAllCaps="false" - android:textAppearance="@style/TextViewNormal" - app:cornerRadius="@dimen/button_corner" app:icon="@drawable/ic_play_arrow_white_24dp" - app:iconGravity="textStart" app:layout_constraintEnd_toStartOf="@+id/shuffleAction" app:layout_constraintHorizontal_bias="0.5" app:layout_constraintStart_toStartOf="parent" @@ -35,14 +29,8 @@ android:layout_marginTop="8dp" android:layout_marginEnd="@dimen/button_margin_horizontal" android:layout_marginBottom="8dp" - android:paddingTop="@dimen/button_padding_vertical" - android:paddingBottom="@dimen/button_padding_vertical" android:text="@string/shuffle" - android:textAllCaps="false" - android:textAppearance="@style/TextViewNormal" - app:cornerRadius="@dimen/button_corner" app:icon="@drawable/ic_shuffle_white_24dp" - app:iconGravity="textStart" app:layout_constraintBottom_toBottomOf="@+id/playAction" app:layout_constraintEnd_toEndOf="parent" app:layout_constraintHorizontal_bias="0.5" diff --git a/app/src/main/res/layout/activity_artist_content.xml b/app/src/main/res/layout/activity_artist_content.xml index d5b60e303..adabb1d51 100644 --- a/app/src/main/res/layout/activity_artist_content.xml +++ b/app/src/main/res/layout/activity_artist_content.xml @@ -13,14 +13,8 @@ android:layout_height="wrap_content" android:layout_marginStart="@dimen/button_margin_horizontal" android:layout_marginEnd="@dimen/button_margin_horizontal" - android:paddingTop="@dimen/button_padding_vertical" - android:paddingBottom="@dimen/button_padding_vertical" android:text="@string/action_play_all" - android:textAllCaps="false" - android:textAppearance="@style/TextViewNormal" - app:cornerRadius="@dimen/button_corner" app:icon="@drawable/ic_play_arrow_white_24dp" - app:iconGravity="textStart" app:layout_constraintEnd_toStartOf="@+id/shuffleAction" app:layout_constraintHorizontal_bias="0.5" app:layout_constraintStart_toStartOf="parent" @@ -32,14 +26,8 @@ android:layout_height="wrap_content" android:layout_marginStart="@dimen/button_margin_horizontal" android:layout_marginEnd="@dimen/button_margin_horizontal" - android:paddingTop="@dimen/button_padding_vertical" - android:paddingBottom="@dimen/button_padding_vertical" android:text="@string/shuffle" - android:textAllCaps="false" - android:textAppearance="@style/TextViewNormal" - app:cornerRadius="@dimen/button_corner" app:icon="@drawable/ic_shuffle_white_24dp" - app:iconGravity="textStart" app:layout_constraintBottom_toBottomOf="@+id/playAction" app:layout_constraintEnd_toEndOf="parent" app:layout_constraintHorizontal_bias="0.5" @@ -85,6 +73,7 @@ android:nestedScrollingEnabled="false" app:layout_constraintStart_toStartOf="parent" app:layout_constraintTop_toBottomOf="@id/albumTitle" + tools:itemCount="3" tools:layoutManager="androidx.recyclerview.widget.GridLayoutManager" tools:listitem="@layout/item_album_card" tools:spanCount="3" /> diff --git a/app/src/main/res/layout/activity_license.xml b/app/src/main/res/layout/activity_license.xml index dac51f10b..45029315b 100644 --- a/app/src/main/res/layout/activity_license.xml +++ b/app/src/main/res/layout/activity_license.xml @@ -2,7 +2,6 @@ diff --git a/app/src/main/res/layout/fragment_color_player.xml b/app/src/main/res/layout/fragment_color_player.xml index 6b289cf1d..4db23c9ac 100644 --- a/app/src/main/res/layout/fragment_color_player.xml +++ b/app/src/main/res/layout/fragment_color_player.xml @@ -27,7 +27,7 @@ - - + android:layout_height="match_parent" + android:scaleType="centerCrop" + tools:ignore="ContentDescription,UnusedAttribute" + tools:srcCompat="@tools:sample/backgrounds/scenic[10]" /> - - - - - - - - - - - - - - - + @@ -31,7 +30,6 @@ diff --git a/app/src/main/res/layout/fragment_mini_player.xml b/app/src/main/res/layout/fragment_mini_player.xml index 8dd6b02aa..41dcf84dc 100644 --- a/app/src/main/res/layout/fragment_mini_player.xml +++ b/app/src/main/res/layout/fragment_mini_player.xml @@ -7,7 +7,7 @@ android:layout_height="48dp" android:background="?colorSecondary" android:clickable="true" - android:focusable="false" + android:focusable="true" tools:ignore="UnusedAttribute"> + app:icon="@drawable/ic_play_arrow_white_24dp" /> + app:icon="@drawable/ic_shuffle_white_24dp" /> \ No newline at end of file diff --git a/app/src/main/res/layout/item_song.xml b/app/src/main/res/layout/item_song.xml index 679adbbf6..24b39e27a 100755 --- a/app/src/main/res/layout/item_song.xml +++ b/app/src/main/res/layout/item_song.xml @@ -27,6 +27,7 @@ android:fontFeatureSettings="onum" android:gravity="center" android:maxLines="1" + tools:text="@tools:sample/date/hhmm" android:minHeight="40dp" android:textAppearance="@style/TextViewNormal" android:textColor="?colorOnSecondary" /> @@ -45,7 +46,7 @@ app:layout_constraintHorizontal_bias="0.5" app:layout_constraintStart_toEndOf="@+id/imageContainer" app:layout_constraintTop_toTopOf="parent" - tools:text="Song title" /> + tools:text="@tools:sample/lorem/random" /> + tools:text="@tools:sample/date/hhmm" /> + app:strokeWidth="0dp"> #607d8b #f5f5f5 #3D5AFE - #212121 - #121212 - #2C2D30 - #121212 + #202124 + #17181a + #222326 + #202124 #FFFFFF #202020 @@ -16,15 +16,17 @@ #FFFFFF #FEFEFE - #121212 - #181818 - #333333 - #1F1F1F + #202124 + #ffffff + #202124 + #121212 + #3C4043 #000000 + #ffffff #000000 #000000 - #121212 + #17181a #00000000 diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml index 713d6170e..66bd544a1 100644 --- a/app/src/main/res/values/styles.xml +++ b/app/src/main/res/values/styles.xml @@ -161,4 +161,17 @@ 8dp + + + + + diff --git a/app/src/main/res/values/styles_parents.xml b/app/src/main/res/values/styles_parents.xml index ab8f24af2..da04a7f01 100644 --- a/app/src/main/res/values/styles_parents.xml +++ b/app/src/main/res/values/styles_parents.xml @@ -27,7 +27,7 @@ @style/ThemeOverlay.AppCompat - @color/dark_color + @color/md_red_500 @style/Widget.ActionButton.Overflow @@ -40,6 +40,7 @@ false @style/MaterialAlertDialogTheme + @style/MaterialButtonTheme @color/darkColorPrimary @color/darkColorSecondary @@ -49,7 +50,7 @@ @color/md_deep_purple_A200 - @color/darkColorBackground + @color/md_red_500 @color/darkColorSurface @color/mi_text_color_primary_dark @color/mi_text_color_secondary_dark @@ -124,8 +125,9 @@ @font/circular false + @style/MaterialAlertDialogTheme - + @style/MaterialButtonTheme @color/lightColorPrimary @color/lightColorSecondary @@ -167,4 +169,14 @@ + + \ No newline at end of file diff --git a/appthemehelper/src/main/java/code/name/monkey/appthemehelper/common/ATHToolbarActivity.java b/appthemehelper/src/main/java/code/name/monkey/appthemehelper/common/ATHToolbarActivity.java index c29b1c3ab..61a9f713d 100755 --- a/appthemehelper/src/main/java/code/name/monkey/appthemehelper/common/ATHToolbarActivity.java +++ b/appthemehelper/src/main/java/code/name/monkey/appthemehelper/common/ATHToolbarActivity.java @@ -1,9 +1,10 @@ package code.name.monkey.appthemehelper.common; import android.graphics.drawable.ColorDrawable; +import android.view.Menu; + import androidx.annotation.Nullable; import androidx.appcompat.widget.Toolbar; -import android.view.Menu; import code.name.monkey.appthemehelper.ATHActivity; import code.name.monkey.appthemehelper.util.ToolbarContentTintHelper; @@ -12,6 +13,15 @@ import code.name.monkey.appthemehelper.util.ToolbarContentTintHelper; public class ATHToolbarActivity extends ATHActivity { private Toolbar toolbar; + public static int getToolbarBackgroundColor(Toolbar toolbar) { + if (toolbar != null) { + if (toolbar.getBackground() instanceof ColorDrawable) { + return ((ColorDrawable) toolbar.getBackground()).getColor(); + } + } + return 0; + } + @Override public boolean onCreateOptionsMenu(Menu menu) { Toolbar toolbar = getATHToolbar(); @@ -34,13 +44,4 @@ public class ATHToolbarActivity extends ATHActivity { protected Toolbar getATHToolbar() { return toolbar; } - - public static int getToolbarBackgroundColor(Toolbar toolbar) { - if (toolbar != null) { - if (toolbar.getBackground() instanceof ColorDrawable) { - return ((ColorDrawable) toolbar.getBackground()).getColor(); - } - } - return 0; - } } \ No newline at end of file