From c0473c4eae2fc0c6e97db8bbf659c53083854666 Mon Sep 17 00:00:00 2001
From: Martin Felber <felber.martin3@gmail.com>
Date: Tue, 26 Nov 2024 20:08:49 +0100
Subject: [PATCH] Addressing small change requests

---
 .../shared/ui/profile_picture/ProfilePicture.kt     | 13 +++++++------
 .../shared/ui/profile_picture/ProfilePictureData.kt |  3 +++
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/feature/metis/shared/src/main/kotlin/de/tum/informatics/www1/artemis/native_app/feature/metis/shared/ui/profile_picture/ProfilePicture.kt b/feature/metis/shared/src/main/kotlin/de/tum/informatics/www1/artemis/native_app/feature/metis/shared/ui/profile_picture/ProfilePicture.kt
index 42e35c48..353203c0 100644
--- a/feature/metis/shared/src/main/kotlin/de/tum/informatics/www1/artemis/native_app/feature/metis/shared/ui/profile_picture/ProfilePicture.kt
+++ b/feature/metis/shared/src/main/kotlin/de/tum/informatics/www1/artemis/native_app/feature/metis/shared/ui/profile_picture/ProfilePicture.kt
@@ -1,6 +1,5 @@
 package de.tum.informatics.www1.artemis.native_app.feature.metis.shared.ui.profile_picture
 
-import android.graphics.drawable.Drawable
 import androidx.compose.foundation.Image
 import androidx.compose.foundation.background
 import androidx.compose.foundation.layout.Box
@@ -29,6 +28,9 @@ import de.tum.informatics.www1.artemis.native_app.core.ui.remote_images.ProfileP
 const val TEST_TAG_PROFILE_PICTURE_IMAGE = "TEST_TAG_PROFILE_PICTURE_IMAGE"
 const val TEST_TAG_PROFILE_PICTURE_INITIALS = "TEST_TAG_PROFILE_PICTURE_INITIALS"
 
+private const val BoxSizeToFontSizeMultiplier = 0.16f
+
+
 @Composable
 fun ProfilePicture(
     modifier: Modifier,
@@ -36,6 +38,7 @@ fun ProfilePicture(
     profilePictureImageProvider: ProfilePictureImageProvider?,
 ) {
     // TODO: Add onClick that opens a dialog with info about the user, see iOS
+    //       https://github.com/ls1intum/artemis-android/issues/154
     when(profilePictureData) {
         is ProfilePictureData.Image -> {
             if (profilePictureImageProvider != null) {
@@ -75,9 +78,7 @@ fun ProfilePictureImage(
         )
     }
 
-    val resultData = loadAsyncImageDrawable(request = request)
-
-    when (resultData.dataState) {
+    when (val resultDataState = loadAsyncImageDrawable(request = request).dataState) {
         is DataState.Failure -> {
             InitialsPlaceholder(
                 modifier = modifier,
@@ -91,7 +92,7 @@ fun ProfilePictureImage(
             )
         }
         is DataState.Success -> {
-            val loadedDrawable = (resultData.dataState as DataState.Success<Drawable>).data
+            val loadedDrawable = resultDataState.data
 
             val painter = remember(loadedDrawable) {
                 val bitmap = loadedDrawable.toBitmap().asImageBitmap()
@@ -129,7 +130,7 @@ fun InitialsPlaceholder(
         Text(
             text = profilePictureData.initials,
             color = Color.White,
-            fontSize = boxSize.intValue.sp.nonScaledSp * 0.16f,
+            fontSize = boxSize.intValue.sp.nonScaledSp * BoxSizeToFontSizeMultiplier,
             fontWeight =  FontWeight.Bold
         )
     }
diff --git a/feature/metis/shared/src/main/kotlin/de/tum/informatics/www1/artemis/native_app/feature/metis/shared/ui/profile_picture/ProfilePictureData.kt b/feature/metis/shared/src/main/kotlin/de/tum/informatics/www1/artemis/native_app/feature/metis/shared/ui/profile_picture/ProfilePictureData.kt
index 7d9e806e..728c724d 100644
--- a/feature/metis/shared/src/main/kotlin/de/tum/informatics/www1/artemis/native_app/feature/metis/shared/ui/profile_picture/ProfilePictureData.kt
+++ b/feature/metis/shared/src/main/kotlin/de/tum/informatics/www1/artemis/native_app/feature/metis/shared/ui/profile_picture/ProfilePictureData.kt
@@ -24,6 +24,9 @@ sealed class ProfilePictureData {
 }
 
 // The following util functions are copied from the Artemis webapp implementation.
+// Sources:
+// https://github.com/ls1intum/Artemis/blob/fa32c243b568c92aa5e075e8176abdc7c7452444/src/main/webapp/app/utils/text.utils.ts
+// https://github.com/ls1intum/Artemis/blob/fa32c243b568c92aa5e075e8176abdc7c7452444/src/main/webapp/app/utils/color.utils.ts
 
 /**
  * Returns 2 capitalized initials of a given string.