Closed Mindohyeon closed 1 year ago
현재 네트워크 상태를 관리하는 모듈의 네이밍이 Networks
입니다. 그 모듈에서 사용되는 라이브러리의 네임 또한 Networks
입니다.
따라서 모듈의 네이밍을 변경하는게 좋아보이는데, NetworksMonitor
와 같이 변경하는건 어떨까요?
모듈에서 사용하는 NWPathMonitor
클래스에서 착안해봤습니다.
현재 네트워크 상태를 관리하는 모듈의 네이밍이
Networks
입니다. 그 모듈에서 사용되는 라이브러리의 네임 또한Networks
입니다. 따라서 모듈의 네이밍을 변경하는게 좋아보이는데,NetworksMonitor
와 같이 변경하는건 어떨까요?모듈에서 사용하는
NWPathMonitor
클래스에서 착안해봤습니다.
굳 아이디어네요 👍
이런 문제가 있었습니다.
143 해당 PR 에 대한 문제로,
네트워크 상태에 따라 이미지 로딩 속도의 문제가 있었고, 이에 따라 이미지가 버벅거리거나 다른 게시물의 이미지와 섞이는 등의 문제가 있었습니다.
이렇게 해결했습니다.
다운샘플링 전에 이미지를 먼저 nil로 초기화 했습니다. 또한 만약, 다운샘플링이 실패해 completion으로 nil이 전달되면 경고 alert를 통해 사용자에게 알립니다. 이를 통해 이미지가 버벅거리지 않고, 현재 로딩되지 않은 이미지는 회색으로 배경으로 보이게 됩니다.