-
Notifications
You must be signed in to change notification settings - Fork 1
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
[#199][#221] 그룹상세 이미지뷰어 연결 #222
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오호 별도 Activity 로 분리하여 문제를 해결했구나! 고생했어..
수정된 코드 자체적으로는 우려되는 점은 없어보여!
다만.. 현재 데이터를 넘기기 위해서 Serializable 을 사용하고 있는데, 커스텀하지 않은 Serializable 을 사용할 것이라면, Serializable 보단 Parcelable 을 사용하는게 어떨까?
기본적으로 Serializable 이 리플렉션을 사용하고 있어서 Parcelable 에 비해 속도가 느린 것으로 알고 있어서 이왕이면 Parcelable 을 사용하는것도 괜찮을 것 같은데 어떻게 생각해??
@JeonK1 아마 안해본건 아닌데....... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
나는 좋다 지혜야~ 코멘트 하나 남기긴했는데 중요하지 않은 부분이라 일단은 어푸 누를게. 계원이 코멘트만 확인해줘~~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: 이거 디테일에서만 사용되는데 액티비티로 뺀 이유가 있을까?? 컴포저블로 하는건 별로일까..? 뭔가 뒤로가기도 있어서 컴포저블로 하는게 더 자연스러웠던거 같아서..! 스테이터스 바 땜에 그런건감..?
Issue No
Overview (Required)
Screenshot