mwy3055 / MyVoca

ENG-KOR Wordbook App for Android with Jetpack Compose
1 stars 0 forks source link

퀴즈 화면 피그마 디자인에 맞게 변경 #152

Closed easyhooon closed 11 months ago

easyhooon commented 11 months ago
easyhooon commented 11 months ago

맞았습니다!!, 틀렸습니다 다이얼로그 내에 전체 뜻을 보여주는 것에 대해선 정책이 정해지면 수정하겠습니다

easyhooon commented 11 months ago

정답/오답 팝업이 떴을 때 뒷 화면이 검게 처리되어야 합니다. 기술적으로 불가능한가요? Compose Dialog를 전체 화면에 띄우면 될 것 같은데 한번 확인해 주세요

https://stackoverflow.com/questions/76551598/jetpack-compose-dialog-not-showing-gray-overlay 이거 에뮬레이터에서 뒤에 딤 처리가 안되는 이슈가 있더군요 저도 그래서 찾아봤는데 실기기에서는 정상적으로 딤처리가 됩니다

easyhooon commented 11 months ago

화면 맨 하단 VersusView의 삼각형이 약간 어긋나 있습니다. 수정 가능할까요?

허걱 핸드폰 마다 다르게 보이는건가.. 확인해보겠습니다!

easyhooon commented 11 months ago

WordMeaning에서 단어 품사와 뜻 텍스트에 MaterialTheme.typography.bodyLarge가 적용되어야 합니다. 사실 퀴즈 단어도 디자인에서 사용한 텍스트 스타일이 적용되지 않았습니다. 수정 부탁드려요. (찾아보니 QuizOption에서 CompositionLocal로 값을 주고 있네요)

https://developer.android.com/jetpack/compose/designsystems/material3?hl=ko 아 공식문서에 각각 크기가 얼만지 명시되어있었군여 이거 보고 고쳐보겠슴니다

mwy3055 commented 11 months ago

https://developer.android.com/jetpack/compose/designsystems/material3?hl=ko 아 공식문서에 각각 크기가 얼만지 명시되어있었군여 이거 보고 고쳐보겠슴니다

공식 문서가 아니라 Figma에 선언되어 있습니다. 피그마의 Material 3 Design Kit for MyVoca 파일에서 확인하실 수 있습니다.

image

easyhooon commented 11 months ago

4번을 QuizScreen 내에 하드 코딩 된 textSize MaterialTheme.typography textStyle 로 변경 에서 수정했습니다

easyhooon commented 11 months ago

4번을 QuizScreen 내에 하드 코딩 된 textSize MaterialTheme.typography textStyle 로 변경 에서 수정했습니다

해당 커밋에서 VersusElement 내부의 텍스트도 MaterialTheme.typography 의 것으로 대체하였더니 Canvas 의 삐져 나온 것이 사라졌습니다(?) 쨌든 1번도 수정했습니다