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

DEP-195 ui: 이미지가 있는 스낵바 구현 #70

Merged
merged 2 commits into from
Dec 15, 2022

Conversation

juhwankim-dev
Copy link
Member

@juhwankim-dev juhwankim-dev commented Dec 12, 2022

💁‍♂️ 변경 내용

KakaoTalk_20221213_021148091

📢 전달사항

기존의 커스텀 스낵바를 살짝 변경 시켜 만들었습니다.
하나로 통일해서 유동적으로 쓰고 싶었는데
할게 너무 많이 남아서 ㅋㅋㅋㅋㅠ 생각할 시간에 UI 하나 더 짜자 싶어서 따로 만들었습니다...
촉박하네요... 화이팅...

@juhwankim-dev juhwankim-dev added the enhancement New feature or request label Dec 12, 2022
@juhwankim-dev juhwankim-dev requested a review from a team as a code owner December 12, 2022 17:14
@juhwankim-dev juhwankim-dev self-assigned this Dec 12, 2022
Copy link
Member

@junhaesung junhaesung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

layoutParams 사용하는 부분에서 lint 가 에러라고 하네요.
배포시 린트를 무시하거나, 코드 고치는 방법이 있을거같은데요.. 어떻게 하는게 좋을까요?

나중에 플레이스토어 심사할 때 lint error 발생하는 것도 심사기준으로 들어가나요?
안들어간다면.. lint 만 살짝 무시하는 방법도ㅋㅋㅋ


val params = layoutParams as FrameLayout.LayoutParams
params.gravity = Gravity.TOP
layoutParams = params
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

요거 지우면 lint error 도 해결되고, gravitiy 옵션도 적용됩니당!

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@juhwankim-dev juhwankim-dev merged commit bc92b3e into develop Dec 15, 2022
@juhwankim-dev juhwankim-dev deleted the feature/DEP-195_custom_snackbar branch December 21, 2022 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants