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.

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

[Feature] 새 이벤트 추가 UI #289

Draft
wants to merge 16 commits into
base: develop
Choose a base branch
from

Conversation

ThirFir
Copy link
Contributor

@ThirFir ThirFir commented Jun 15, 2024

개요

Business 앱 새 이벤트 추가 Feature (Only Presentation Layer : View, ViewModel)


세부 작업 내용

  • Photo Picker 사용(요구 API 30)
    • API 30 미만 : ACTION_OPEN_DOCUMENT로 사진 선택 수행

  • LinedTextField, LinedAlertTextField Composable 정의
    • 기존에 이미 LinedTextField 존재. 우선 LinedTextField2라는 이름으로 정의 -> 추후 협의 및 변경 필요
    • LinedTextField 안에 Alert Row를 두지 않고 두 개의 Composable로 분류한 이유
      1. Alert가 보이지 않는 상태더라도 공간을 차지도록 하기 위함
      2. 날짜 입력 TextField들(startYear, startMonth, ...)은 Alert 상태로 변하더라도 border 색상만 바뀔 뿐, 각자가 Alert 문구를 띄워선 안됨. LinedTextField가 Alert Row를 포함할 경우 이런 Case를 막기 위한 불필요한 변수(ex. isAlertRowAvailable) 하나가 더 필요하였음.

[LinedTextField2]
image

image

[LinedAlertTextField]
image

  • FocusRequester
    • 날짜 입력 완료 시(글자수 채우기로 판단) 자동으로 다음 날짜 입력으로 이동

해결하지 못한 것

image
  • X버튼의 클릭 범위가 보이는 이미지보다 큼.
  • Vector이미지의 크기도 16dp/16dp 이고 Modifier.size(16.dp).clickable{ ... }로 설정했음에도 보이는 크기보다 클릭 범위가 더 넓음

결과

image

@ThirFir ThirFir requested a review from a team as a code owner June 15, 2024 10:18
@ThirFir ThirFir self-assigned this Jun 15, 2024
@BCSDLab BCSDLab locked as off-topic and limited conversation to collaborators Sep 30, 2024
@ThirFir ThirFir marked this pull request as draft September 30, 2024 03:22
@ThirFir ThirFir removed their assignment Sep 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant