From da5968188f8313b6df4a6b53b54859022fa58a87 Mon Sep 17 00:00:00 2001
From: Ahmed El-Helw <ahmedre@gmail.com>
Date: Sun, 1 Dec 2024 18:27:36 +0400
Subject: [PATCH] Fix audio bar in RTL

The audio bar should always be LTR, otherwise the icons will be facing
the wrong direction in the RTL layout.
---
 .../feature/audiobar/ui/PlaybackAudioBar.kt   | 89 ++++++++++---------
 1 file changed, 47 insertions(+), 42 deletions(-)

diff --git a/feature/audiobar/src/main/kotlin/com/quran/mobile/feature/audiobar/ui/PlaybackAudioBar.kt b/feature/audiobar/src/main/kotlin/com/quran/mobile/feature/audiobar/ui/PlaybackAudioBar.kt
index ccde85894b..c9583ebf84 100644
--- a/feature/audiobar/src/main/kotlin/com/quran/mobile/feature/audiobar/ui/PlaybackAudioBar.kt
+++ b/feature/audiobar/src/main/kotlin/com/quran/mobile/feature/audiobar/ui/PlaybackAudioBar.kt
@@ -13,10 +13,13 @@ import androidx.compose.material.icons.filled.Stop
 import androidx.compose.material3.Icon
 import androidx.compose.material3.IconButton
 import androidx.compose.runtime.Composable
+import androidx.compose.runtime.CompositionLocalProvider
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
+import androidx.compose.ui.platform.LocalLayoutDirection
 import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.tooling.preview.Preview
+import androidx.compose.ui.unit.LayoutDirection
 import com.quran.labs.androidquran.common.ui.core.QuranIcons
 import com.quran.labs.androidquran.common.ui.core.QuranTheme
 import com.quran.mobile.feature.audiobar.state.AudioBarState
@@ -57,56 +60,58 @@ internal fun AudioBar(
   modifier: Modifier = Modifier,
   actionButton: @Composable () -> Unit
 ) {
-  Row(
-    horizontalArrangement = Arrangement.SpaceEvenly,
-    verticalAlignment = Alignment.CenterVertically,
-    modifier = modifier
-  ) {
-    IconButton(onClick = { sink(AudioBarUiEvent.CommonPlaybackEvent.Stop) }) {
-      Icon(QuranIcons.Stop, contentDescription = "")
-    }
+  CompositionLocalProvider(LocalLayoutDirection provides LayoutDirection.Ltr) {
+    Row(
+      horizontalArrangement = Arrangement.SpaceEvenly,
+      verticalAlignment = Alignment.CenterVertically,
+      modifier = modifier
+    ) {
+      IconButton(onClick = { sink(AudioBarUiEvent.CommonPlaybackEvent.Stop) }) {
+        Icon(QuranIcons.Stop, contentDescription = "")
+      }
 
-    IconButton(onClick = { sink(AudioBarUiEvent.CommonPlaybackEvent.Rewind) }) {
-      Icon(QuranIcons.FastRewind, contentDescription = "")
-    }
+      IconButton(onClick = { sink(AudioBarUiEvent.CommonPlaybackEvent.Rewind) }) {
+        Icon(QuranIcons.FastRewind, contentDescription = "")
+      }
 
-    actionButton()
+      actionButton()
 
-    IconButton(onClick = { sink(AudioBarUiEvent.CommonPlaybackEvent.FastForward) }) {
-      Icon(QuranIcons.FastForward, contentDescription = "")
-    }
+      IconButton(onClick = { sink(AudioBarUiEvent.CommonPlaybackEvent.FastForward) }) {
+        Icon(QuranIcons.FastForward, contentDescription = "")
+      }
 
-    val infinity = stringResource(id = com.quran.mobile.common.ui.core.R.string.infinity)
-    RepeatableButton(
-      icon = QuranIcons.Repeat,
-      contentDescription = "",
-      values = REPEAT_VALUES,
-      value = state.repeat,
-      defaultValue = 0,
-      format = {
-        if (it > -1) {
-          it.toString()
-        } else {
-          infinity
+      val infinity = stringResource(id = com.quran.mobile.common.ui.core.R.string.infinity)
+      RepeatableButton(
+        icon = QuranIcons.Repeat,
+        contentDescription = "",
+        values = REPEAT_VALUES,
+        value = state.repeat,
+        defaultValue = 0,
+        format = {
+          if (it > -1) {
+            it.toString()
+          } else {
+            infinity
+          }
         }
+      ) {
+        sink(AudioBarUiEvent.CommonPlaybackEvent.SetRepeat(it))
       }
-    ) {
-      sink(AudioBarUiEvent.CommonPlaybackEvent.SetRepeat(it))
-    }
 
-    RepeatableButton(
-      icon = QuranIcons.Speed,
-      contentDescription = "",
-      values = SPEED_VALUES,
-      value = state.speed,
-      defaultValue = 1.0f,
-      format = { it.toString() }
-    ) {
-      sink(AudioBarUiEvent.CommonPlaybackEvent.SetSpeed(it))
-    }
+      RepeatableButton(
+        icon = QuranIcons.Speed,
+        contentDescription = "",
+        values = SPEED_VALUES,
+        value = state.speed,
+        defaultValue = 1.0f,
+        format = { it.toString() }
+      ) {
+        sink(AudioBarUiEvent.CommonPlaybackEvent.SetSpeed(it))
+      }
 
-    IconButton(onClick = { sink(AudioBarUiEvent.CommonPlaybackEvent.ShowSettings) }) {
-      Icon(QuranIcons.Settings, contentDescription = "")
+      IconButton(onClick = { sink(AudioBarUiEvent.CommonPlaybackEvent.ShowSettings) }) {
+        Icon(QuranIcons.Settings, contentDescription = "")
+      }
     }
   }
 }