search

sabujak-sabujak / Pickle

Instagram-oriented android image picker library
Apache License 2.0
4 stars 2 forks source link

InstaFragment 의 toolbar 제거 #26

Closed kklyoon closed 4 years ago

kklyoon commented 4 years ago

오늘 말씀드린대로 샘플앱의 액션바를 살리고 InstaFragment 의 toolbar 를 제거하고 view 로 교체하였습니다.

교체하면서 나온 버그 ( preview 에서 y 좌표 밀리는 현상 )

기존의 버그 ( 다중 선택시 순서 안맞는 문제, count 집계 오류 ) 등을 수정하였습니다.

기존의 OptionMenuViewModel 클래스를 필요한 코드만 남기고 InstaTopViewModel 로 바꿨습니다.

그리고 Inspect Code 돌려서 warning 드는 것들을 소소하게 고쳐보았습니다.

Charlezz commented 4 years ago

구조를 개선하다 보니 Conflict가 발생하는군요...제가 해결하려했으나 풀리퀘 정책상 요청한 쪽에서 해결하는게 기본인것 같습니다.(죄송 ㅠㅠ) 아무래도 베타 릴리즈 이전까지는 공통된 기능도 코드를 두 벌로 가져가야 서로에게 득이 될거 같습니다. 리뷰 없이 작업하시기 편하라고 Write권한을 임시로 드렸습니다.