Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

단어 목록 화면 피그마 디자인 반영 및 버그 수정 #154

Closed
wants to merge 64 commits into from
Closed

단어 목록 화면 피그마 디자인 반영 및 버그 수정 #154

wants to merge 64 commits into from

Conversation

easyhooon
Copy link
Collaborator

  • 피그마 디자인 반영
  • 단어 목록 화면에서 수정하기 버튼을 통해 단어 추가 화면으로 이동할 때 앱이 죽던 문제를 해결
  • 뷰모델 내에 Context 참조 제거

mwy3055 and others added 30 commits November 10, 2022 13:17
@mwy3055
gradle: update dependencies
613d09c
@mwy3055
Merge remote-tracking branch 'origin/develop' into develop
9d1157d 
# Conflicts:
#	app/build.gradle
#	build.gradle
@easyhooon
fix: google-services.json 파일 수정
faa501c 
debug 환경에서 빌드가 가능하도록
@easyhooon
gradle: release, debug 앱 분리
0d5fdde
@easyhooon
Merge pull request #139 from mwy3055/fix/debug-build
59d78ce 
debug 에서 빌드가 가능하도록 설정 변경
@easyhooon
library version update
c813bc7
@easyhooon
Theme Material3 로 변경
3c695e0
@easyhooon
gitignore update
9399c34
@easyhooon
gitignore update 반영
55d1642
@easyhooon
BaselineProfileGenerator 실행 관련 에러 해결
877d00e 
AGP 8.1 버전에 맞는 코드로 변경 및 proguard-rules.pro 파일 수정
@easyhooon
BackdropScaffold 를 제외한 Component, Screen Material3 로 migration
d0781d5 
BackdropScaffold 는 Material3 에서는 아직 사용할 수 없는 API 이므로 다른 Component 로 대체 해야함
-> UI 가 변경 되었으므로 이를 ModalBottomSheet 로 구현할 예정
@mwy3055
gradle: jacoco 업데이트 대응
e43c14f
@mwy3055
Merge remote-tracking branch 'origin/refactor/m3-migration' into refa…
3832fcb 
…ctor/m3-migration
@easyhooon
Merge pull request #140 from mwy3055/refactor/m3-migration
50f1228 
Material3로 migration
@easyhooon
MyVocaText Composable 추가
a9ee076 
M3 로 이전하면서 defaultFontFamily 옵션이 사라져, 직접 font를 지정 해주어야 하는데, 이는 이전에 PR 에서 처럼 몇몇 Text에 대해 누락 시킬 가능성이 존재하므로(휴먼 에러) 디폴트로 Font가 지정된 MyVocaText 로 Text 를 대체
@easyhooon
Accompanist WebView 의존성 추가
bdae679
@easyhooon
단어 추가 화면 내에 웹뷰 연동
cf157b6
@easyhooon
검색 버튼을 눌렀을 때 네이버 사전에 해당 단어가 검색된 화면이 출력 되도록 구현
ed42adb
@easyhooon
uiState collectAsState -> collectAsStateWithLifecycle 로 변경
4e670de 
lifecycle-compose-runtime 라이브러리 의존성 추가
@easyhooon
단어를 변경 하고 나서 검색 버튼을 누르면 웹뷰가 갱신 되도록 구현
2bde9a0
@easyhooon
TopAppBar 그림자 추가
ad43249
@easyhooon
BackdropScaffold -> ModalBottomSheet Migration
83dd90b
@easyhooon
TopBar 의 완료 버튼을 통해 웹뷰를 닫는 기능 구현
bf18802
@easyhooon
검색 버튼을 눌렀을 때 키보드가 자동으로 내려가도록 구현
d7fa2d2
@easyhooon
BackdropScaffold 롤백, Snackbar 실행 취소 버튼 추가
4bc5fd6
@easyhooon
BackdropScaffold frontLayerContent 가 conceal 될 때 content color 가 투명 해…
55b30c9 
…지는 속성 제거
@easyhooon
jacoco version 통일
f6bed3a
@easyhooon
Merge pull request #141 from mwy3055/feature/add-webview
4a27c01 
단어 검색 화면 내에 웹뷰 추가
@easyhooon
BackdropScaffold appBar QueryHeader 제거
056465c
@easyhooon
Merge remote-tracking branch 'origin/develop' into feaure/snackbar-wi…
e109c78 
…th-undo
easyhooon and others added 24 commits November 7, 2023 02:08
@easyhooon
각 화면 내에 뷰모델 주입 방법 통일
508239e 
각 컴포저블 스크린의 생명 주기의 관리 문제로 snackbar 가 다른 화면으로 이동 후 다시 돌아오는 경우 재호출 되는 문제 해결

deprecated 된 아이콘 교체
@easyhooon
Merge remote-tracking branch 'origin/develop' into feaure/snackbar-wi…
999b339 
…th-undo

# Conflicts:
#	app/src/main/java/hsk/practice/myvoca/ui/screens/allword/AllWord.kt
#	app/src/main/java/hsk/practice/myvoca/ui/screens/home/HomeScreen.kt
#	app/src/main/java/hsk/practice/myvoca/ui/screens/profile/ProfileScreen.kt
@easyhooon
하드 코딩된 텍스트 string resource 로 추출
2f508ca
@easyhooon
Merge pull request #147 from mwy3055/feaure/snackbar-with-undo
c08b294 
Snackbar Undo 기능 구현
@easyhooon
단어 추가 화면 피그마 디자인에 맞게 UI 변경
b8ffba4
@easyhooon
추가할 단어의 의미 중복 검사 기능 추가
7c19ee0
@easyhooon
퀴즈 화면 피그마 디자인에 맞게 변경
9a5b8c1
@easyhooon
디자인 피드백 반영
5582f2e
@easyhooon
Modifier 관련 피드백 반영
8a04562
@mwy3055
refactor: modifier가 없는 composable에 modifier 매개변수 추가
8ea2643
@mwy3055
fix: 단어, 뜻 입력창 뒤에 오는 아이콘을 정확히 중간에 맞추도록 수정
407755e
@mwy3055
fix: 단어, 뜻 입력창의 trailing icon이 에러 상태일 때 클릭 불가능하도록 수정
02a3ec2
@easyhooon
Merge pull request #148 from mwy3055/design/add-word-screen
f49c4a2 
단어 추가 화면 피그마 디자인 반영
@easyhooon
퀴즈 선택지가 중복으로 나오는 상황 제거
196410d
@easyhooon
맞은 퀴즈와 틀린 퀴즈가 존재할 때만 VersusView 의 직각 삼각형이 포개어지도록 조건 추가
d22dc58
@easyhooon
refactor: modifier 가 없는 composable 에 modifier 매개변수 추가
f7f6f41
@easyhooon
Merge remote-tracking branch 'origin/develop' into design/quiz-screen
320238a
@easyhooon
QuizScreen 내에 하드 코딩 된 textSize MaterialTheme.typography textStyle 로 변경
26f642d
@easyhooon
Merge pull request #152 from mwy3055/design/quiz-screen
7c4c728 
퀴즈 화면 피그마 디자인에 맞게 변경
@easyhooon
AllWord 화면 피그마 디자인에 맞게 변경
943620c
@easyhooon
뷰모델 내에 Context 참조 제거
8be5d78 
뷰모델에서 뷰의 context 를 사용하는 것은 권장되지 않음
@easyhooon
AllWordScreen 에서 아이템의 수정하기 버튼을 통해 AddWordScreen 으로 이동할 때 앱이 죽는 문제 해결
5634069 
Word 와 각각의 Meaning 에 Status 를 부여
@easyhooon
검색 결과를 눌러야 총 단어 -> 검색 결과로 텍스트가 바뀌도록 수정
e10a65e 
텍스트가 바뀔 때마다 검색이 되는 것이 아니므로 검색 버튼을 누르는 것으로 상태를 변경
@easyhooon
모호한 변수명 및 판단 로직 수정
9032b77
@easyhooon easyhooon added bug Something isn't working refactor Refactor the code. design tasks related to design aspects of the project labels Nov 28, 2023
@easyhooon easyhooon self-assigned this Nov 28, 2023
@easyhooon easyhooon closed this Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mwy3055 mwy3055 Awaiting requested review from mwy3055

Assignees

@easyhooon easyhooon

Labels
bug Something isn't working design tasks related to design aspects of the project refactor Refactor the code.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@easyhooon @mwy3055