diff --git a/app/src/main/java/com/github/libretube/extensions/ToDp.kt b/app/src/main/java/com/github/libretube/extensions/ToDp.kt deleted file mode 100644 index 5523fee74..000000000 --- a/app/src/main/java/com/github/libretube/extensions/ToDp.kt +++ /dev/null @@ -1,8 +0,0 @@ -package com.github.libretube.extensions - -import android.content.res.Resources - -fun Int.toDp(resources: Resources): Float { - val scale = resources.displayMetrics.density - return this * scale + 0.5f -} diff --git a/app/src/main/java/com/github/libretube/extensions/ToPixel.kt b/app/src/main/java/com/github/libretube/extensions/ToPixel.kt new file mode 100644 index 000000000..1af59c65c --- /dev/null +++ b/app/src/main/java/com/github/libretube/extensions/ToPixel.kt @@ -0,0 +1,10 @@ +package com.github.libretube.extensions + +import android.content.res.Resources + +/** + * Convert DP to pixels + */ +fun Int.toPixel(): Float { + return this * Resources.getSystem().displayMetrics.density + 0.5f +} diff --git a/app/src/main/java/com/github/libretube/ui/adapters/PlaylistBookmarkAdapter.kt b/app/src/main/java/com/github/libretube/ui/adapters/PlaylistBookmarkAdapter.kt index 3dc6be6f6..bbb6c4cb2 100644 --- a/app/src/main/java/com/github/libretube/ui/adapters/PlaylistBookmarkAdapter.kt +++ b/app/src/main/java/com/github/libretube/ui/adapters/PlaylistBookmarkAdapter.kt @@ -7,7 +7,7 @@ import androidx.recyclerview.widget.RecyclerView import com.github.libretube.databinding.PlaylistBookmarkRowBinding import com.github.libretube.db.obj.PlaylistBookmark import com.github.libretube.enums.PlaylistType -import com.github.libretube.extensions.toDp +import com.github.libretube.extensions.toPixel import com.github.libretube.ui.sheets.PlaylistOptionsBottomSheet import com.github.libretube.ui.viewholders.PlaylistBookmarkViewHolder import com.github.libretube.util.ImageHelper @@ -31,7 +31,7 @@ class PlaylistBookmarkAdapter( holder.binding.apply { if (bookmarkMode == BookmarkMode.HOME) { val params = root.layoutParams - params.width = (210).toDp(root.context.resources).toInt() + params.width = (210).toPixel().toInt() root.layoutParams = params } diff --git a/app/src/main/java/com/github/libretube/ui/adapters/VideosAdapter.kt b/app/src/main/java/com/github/libretube/ui/adapters/VideosAdapter.kt index f67dff7fb..5b009b8d2 100644 --- a/app/src/main/java/com/github/libretube/ui/adapters/VideosAdapter.kt +++ b/app/src/main/java/com/github/libretube/ui/adapters/VideosAdapter.kt @@ -17,7 +17,7 @@ import com.github.libretube.databinding.AllCaughtUpRowBinding import com.github.libretube.databinding.TrendingRowBinding import com.github.libretube.databinding.VideoRowBinding import com.github.libretube.extensions.formatShort -import com.github.libretube.extensions.toDp +import com.github.libretube.extensions.toPixel import com.github.libretube.extensions.toID import com.github.libretube.ui.base.BaseActivity import com.github.libretube.ui.extensions.setFormattedDuration @@ -91,8 +91,8 @@ class VideosAdapter( // set a fixed width for better visuals val params = root.layoutParams when (forceMode) { - ForceMode.RELATED -> params.width = (180).toDp(root.context.resources).toInt() - ForceMode.HOME -> params.width = (250).toDp(root.context.resources).toInt() + ForceMode.RELATED -> params.width = (180).toPixel().toInt() + ForceMode.HOME -> params.width = (250).toPixel().toInt() else -> {} } root.layoutParams = params diff --git a/app/src/main/java/com/github/libretube/ui/fragments/LibraryFragment.kt b/app/src/main/java/com/github/libretube/ui/fragments/LibraryFragment.kt index b1b1bb14c..d266180db 100644 --- a/app/src/main/java/com/github/libretube/ui/fragments/LibraryFragment.kt +++ b/app/src/main/java/com/github/libretube/ui/fragments/LibraryFragment.kt @@ -16,7 +16,7 @@ import com.github.libretube.api.PlaylistsHelper import com.github.libretube.constants.PreferenceKeys import com.github.libretube.databinding.FragmentLibraryBinding import com.github.libretube.extensions.TAG -import com.github.libretube.extensions.toDp +import com.github.libretube.extensions.toPixel import com.github.libretube.ui.adapters.PlaylistsAdapter import com.github.libretube.ui.base.BaseFragment import com.github.libretube.ui.dialogs.CreatePlaylistDialog @@ -91,7 +91,7 @@ class LibraryFragment : BaseFragment() { // optimize CreatePlaylistFab bottom margin if miniPlayer active val bottomMargin = if (isMiniPlayerVisible) 64 else 16 val layoutParams = binding.createPlaylist.layoutParams as ViewGroup.MarginLayoutParams - layoutParams.bottomMargin = bottomMargin.toDp(resources).toInt() + layoutParams.bottomMargin = bottomMargin.toPixel().toInt() binding.createPlaylist.layoutParams = layoutParams } diff --git a/app/src/main/java/com/github/libretube/ui/fragments/PlaylistFragment.kt b/app/src/main/java/com/github/libretube/ui/fragments/PlaylistFragment.kt index 3ab9b72e2..cb98de13a 100644 --- a/app/src/main/java/com/github/libretube/ui/fragments/PlaylistFragment.kt +++ b/app/src/main/java/com/github/libretube/ui/fragments/PlaylistFragment.kt @@ -23,7 +23,7 @@ import com.github.libretube.enums.PlaylistType import com.github.libretube.extensions.TAG import com.github.libretube.extensions.awaitQuery import com.github.libretube.extensions.query -import com.github.libretube.extensions.toDp +import com.github.libretube.extensions.toPixel import com.github.libretube.extensions.toID import com.github.libretube.ui.adapters.PlaylistAdapter import com.github.libretube.ui.base.BaseFragment @@ -81,7 +81,7 @@ class PlaylistFragment : BaseFragment() { playerViewModel.isMiniPlayerVisible.observe(viewLifecycleOwner) { binding.playlistRecView.updatePadding( - bottom = if (it) (64).toDp(resources).toInt() else 0 + bottom = if (it) (64).toPixel().toInt() else 0 ) } diff --git a/app/src/main/java/com/github/libretube/ui/fragments/WatchHistoryFragment.kt b/app/src/main/java/com/github/libretube/ui/fragments/WatchHistoryFragment.kt index a74fed717..09856670a 100644 --- a/app/src/main/java/com/github/libretube/ui/fragments/WatchHistoryFragment.kt +++ b/app/src/main/java/com/github/libretube/ui/fragments/WatchHistoryFragment.kt @@ -12,7 +12,7 @@ import androidx.recyclerview.widget.RecyclerView import com.github.libretube.databinding.FragmentWatchHistoryBinding import com.github.libretube.db.DatabaseHolder.Companion.Database import com.github.libretube.extensions.awaitQuery -import com.github.libretube.extensions.toDp +import com.github.libretube.extensions.toPixel import com.github.libretube.ui.adapters.WatchHistoryAdapter import com.github.libretube.ui.base.BaseFragment import com.github.libretube.ui.models.PlayerViewModel @@ -37,7 +37,7 @@ class WatchHistoryFragment : BaseFragment() { playerViewModel.isMiniPlayerVisible.observe(viewLifecycleOwner) { binding.watchHistoryRecView.updatePadding( - bottom = if (it) (64).toDp(resources).toInt() else 0 + bottom = if (it) (64).toPixel().toInt() else 0 ) } diff --git a/app/src/main/java/com/github/libretube/ui/views/CustomExoPlayerView.kt b/app/src/main/java/com/github/libretube/ui/views/CustomExoPlayerView.kt index 690e06c5f..6d73d0f16 100644 --- a/app/src/main/java/com/github/libretube/ui/views/CustomExoPlayerView.kt +++ b/app/src/main/java/com/github/libretube/ui/views/CustomExoPlayerView.kt @@ -14,7 +14,7 @@ import com.github.libretube.databinding.DoubleTapOverlayBinding import com.github.libretube.databinding.ExoStyledPlayerControlViewBinding import com.github.libretube.databinding.PlayerGestureControlsViewBinding import com.github.libretube.extensions.normalize -import com.github.libretube.extensions.toDp +import com.github.libretube.extensions.toPixel import com.github.libretube.obj.BottomSheetItem import com.github.libretube.ui.activities.MainActivity import com.github.libretube.ui.base.BaseActivity @@ -494,8 +494,8 @@ internal class CustomExoPlayerView( super.onConfigurationChanged(newConfig) val offset = when (newConfig?.orientation) { - Configuration.ORIENTATION_LANDSCAPE -> 20.toDp(resources) - else -> 10.toDp(resources) + Configuration.ORIENTATION_LANDSCAPE -> 20.toPixel() + else -> 10.toPixel() } binding.progressBar.let {