DCRefresher icon indicating copy to clipboard operation
DCRefresher copied to clipboard

개선 요청사항 모음

Open seo-rii opened this issue 3 years ago • 7 comments

좋은 프로그램 감사드립니다. 몇 가지 더 있으면 좋을 것 같은 기능이 있어 요청해봅니다.

개선점

  • [x] history api 사용해서 preview시 주소 변경
  • [x] preview시 title 변경
  • [x] 왼쪽 클릭으로도 preview 열 수 있게 하기 Preview 기능이 정말 너무 좋습니다. 댓글만 달 수 있다면 오히려 기본 뷰보다 나은 것 같아 아예 기본 뷰를 대체하는 옵션을 넣으면 좋을 것 같습니다.
  • [x] write_div에 width 대신 max-width 속성 적용
  • [x] refresher-frame 대신 refresher-frame-outer에 블러 효과 적용
  • [x] image에 margin 주기
  • [ ] 이전 게시글/다음 게시글 버튼 만들기
  • [ ] 게시글 내 이미지 전체 다운로드 버튼
  • [ ] 다크 모드를 시스템 설정에 따르게 하도록 옵션 추가
  • [x] 설정 화면 크기 증가
  • [ ] 본문 이미지 클릭했을 시 새 창에서 열지 않고 다운로드 버튼 표시

우선순위 낮음

  • [ ] nsfw.js 적용
  • [ ] 내가 작성한 게시글 일괄 조회
  • [ ] 게시물 북마크에 추가 버튼 만들기
  • [ ] 디시의 불안정한 서버를 감안하여 preview에서 오류로 인한 요청 실패시(삭제가 아닌) 최대 n번 재시도

버그

  • [x] Preview에서 이전/이후 게시글로 이동했을 때 글이 읽은 것으로 표시되지 않음
  • [x] 추천 버튼을 누를 시 추천수 포맷팅(세자리 단위로 쉼표)가 깨지는 문제
  • [ ] 광고 차단시 링크만 없어지고 틀이 없어지지 않음 image
  • [ ] 인디케이터 페이지 전환 옵션이 켜져있어도 위의 메뉴(전체글, 개념글) 클릭시에는 페이지 전환됨
  • [ ] 위쪽(갤러리 설명)에 있는 게시글 및 메인페이지에서는 preview가 작동하지 않음
  • [ ] 내용이 적어서 한 페이지 내에 모든 내용이 들어가는 경우에도 스크롤바가 표시됨

seo-rii avatar Jan 12 '21 04:01 seo-rii

어.. ㅋㅋㅋㅋ 일단 확장 프로그램에 애정?을 가지고 사용해주시는 점 감사드립니다. 몇몇 사항들은 구현하기 조금 그렇기에 구현하기 조금 어려운 부분들은 이유를 들어 설명하겠습니다.

개선점

~~본문과 댓글창 합친것처럼 보이기~~

1.0 버전에서는 본문과 댓글창에 합쳐진 단일 프레임이었습니다. 2.0에 들어 댓글창과 본문 창이 나뉜 이유는 디자인적으로 한 프레임 안에서 로딩 인디케이터가 2개 이상 돌아가는 게 깔끔하지 않아 보이고, 시스템적으로도 한 프레임 안에 본문 정보를 담은 객체와 댓글 정보를 담은 객체가 널브러져 있는 게 이상하게 보여 댓글 창을 분리하게 되었습니다. 말씀하신 합친 것처럼? 보이게 하는 것은 프레임 간격을 줄여 가능이야 하겠지만은, image 디자인적으로 어떻게 해결해야 할지 감이 안 잡히네요.

  • [x] refresher-frame 대신 refresher-frame-outer에 블러 효과 적용
  • [x] write_div에 width 대신 max-width 속성 적용
  • [x] preview시 title 변경

~~이전 게시글/다음 게시글 버튼 만들기~~

미리보기 모듈은 아예 "미리보기" 기능에만 충실했으면 좋겠다고 저는 생각하고 있습니다. 개인적으로 스크롤하여 다음 게시글 보기 기능이 UI를 최대한 복잡하지 않고 보기 좋게 만드는 방법이라 생각하여 이 정도 선이 게시글을 탐색하는 마지노선이라 생각해주시면 감사하겠습니다.

~~이전/다음 게시글 이동시 삭제된 게시글은 건너뛰기~~

이 부분은 시스템적으로 구현이 어렵습니다. 페이지에 있는 게시글 목록을 확인하여 게시글이 존재하는지 확인할 수는 있겠지만, 미리보는 게시글이 다음 페이지에 있는 경우에는 이 방법이 적용 불가능하고, 다음 게시글이 존재하는지 요청을 보내 확인할 수도 있겠으나 서버에 부담을 줄 수 있고, 부담을 줄이기 위해 이를 최대 n번으로 제한하더라도 다음 게시글이 n개만큼 삭제되어 있을 경우 아무 소용이 없기 때문에 구현이 어려울 듯싶습니다.

  • [x] 왼쪽 클릭으로도 preview 열 수 있게 하기 Preview 기능이 정말 너무 좋습니다. 댓글만 달 수 있다면 오히려 기본 뷰보다 나은 것 같아 아예 기본 뷰를 대체하는 옵션을 넣으면 좋을 것 같습니다.

이 부분은 원래는 기존 링크로 이동하는 액션과 헷갈려 넣지 않으려고 했으나 많은 분들이 요청을 주시기 때문에 오른쪽 클릭과 왼쪽 클릭의 기능을 반전시키는 방식으로 한 번 넣어보겠습니다.

~~추천/비추를 눌렀을 때 표시하기~~

미리보기 창이 닫힐 경우 데이터가 날아가서 소용이 없을 듯 싶습니다. 상태를 저장한다 하더라도 굳이?라는 생각이 드네요.

  • [ ] 게시글 내 이미지 전체 다운로드 버튼
  • [ ] nsfw.js 적용

모델 크기가 커서 활성화된 경우 외부 리소스를 fetch 하는 방식으로 추후에 구현해보도록 하겠습니다.

~~내가 작성한 게시글 일괄 검색/삭제~~

클리너 기능은 디시에서 금방 막기에 구현이 어려울 듯싶습니다.

  • [x] Preview 창에서 댓글 달 수 있도록 추가

~~추천/비추 자동 업데이트~~

추천/비추 수를 받아오려면 페이지를 주기적으로 고쳐줘야 하는데, 이럴 경우 게시글 조회수가 계속 증가하는 문제가 발생하여 구현이 어렵습니다.

~~목록에서 맨 아래에서 한번 더 스크롤시 다음 번호로 이동(preview같이)~~

목록 아래에도 검색창, 갤러리 링크 등의 컨텐츠들이 있기 때문에 사용에 불편함이 따를 것 같습니다.

  • [ ] 게시물 북마크에 추가 버튼 만들기
  • [ ] 대댓글에서 댓글이랑 같은 아이디/IP 발견시 하이라이트

버그

  • [x] 광고 차단시 링크만 없어지고 틀이 없어지지 않음 image
  • [ ] 디시콘 보기 버튼을 눌렀을때 반응 없음

해당 버그는 인지하고 있었으며 추후에 디시콘을 미리보는 기능과 함께 수정하도록 하겠습니다.

+추가

개선점

  • [x] history api 사용해서 preview시 주소 변경

"미리보기" 모듈의 이름 그대로 미리 보는 용도로 제작하였지만 옵션으로 추가해보겠습니다.

  • [ ] 다크 모드를 시스템 설정에 따르게 하도록 옵션 추가

~~목록에서 추천 옆에 비추천 횟수 표시~~

시스템적으로 구현이 어렵습니다. 이유는 추천/비추 자동 업데이트 의 이유와 같습니다.

  • [x] 설정 화면 크기 증가

~~preview 화면에 신고 버튼 추가~~

메이저 갤러리 관리진들이 게시글을 모니터링하여 자르고, 마이너, 미니 갤러리의 경우 신고 기능이 아닌 자체적으로 불판을 만들어 운영하기 때문에 실효성이 있을까 걱정이 됩니다. 그리고 디자인적으로도 버튼이 너무 많아질 것 같습니다.

  • [x] 다음 뉴스처럼 새 댓글이 달렸을때 새로고침 버튼 표시

새 댓글이 달렸을 때 는 어렵고 항시 자리하고 있는 새로고침 버튼은 넣어보도록 하겠습니다.

~~게시글 내용 전체 복사 버튼 추가~~

사용 빈도에 비해 항상 버튼을 차지하는 것이 디자인적으로 별로일 것 같습니다.

  • [ ] 본문 이미지 클릭했을 시 새 창에서 열지 않고 다운로드 버튼 표시

버그

  • [ ] 인디케이터 페이지 전환 옵션이 켜져있어도 위의 메뉴(전체글, 개념글) 클릭시에는 페이지 전환됨
  • [ ] 위쪽(갤러리 설명)에 있는 게시글 및 메인페이지에서는 preview가 작동하지 않음
  • [ ] 내용이 적어서 한 페이지 내에 모든 내용이 들어가는 경우에도 스크롤바가 표시됨

So-chiru avatar Jan 12 '21 06:01 So-chiru

하나하나 검토해주셔서 너무 감사합니다! 추가 의견 드려봅니다.

  • 이전글/다음글 버튼 이런 식으로 버튼을 넣으면 UI의 복잡성을 피하면서도 이전/다음 버튼을 넣을 수 있을 것 같은데 어떻게 생각하세요? image
  • 본문과 댓글창 합진것처럼 보이기 먼저 width를 일치하게 하고 refresher-frame 개수에 위치에 따라 border-radius를 다르게 주어서 시작/끝에만 border-radius를 주면 될 것 같습니다. 그림자는 이걸 참고해보세요.
  • 내가 작성한 게시글 일괄 검색/삭제 클리너 기능이 어렵다면 자신이 작성한 글을 단순히 저장하고 이를 볼 수 있도록 하는 것은 가능할까요? (로그인 안 한 경우에도 작성한 글 목록을 볼 수 있도록)

seo-rii avatar Jan 12 '21 06:01 seo-rii

저가 코드를 다 읽어보지 못해서 깊숙한 수정은 어렵지만 preview시 title, url 수정하는것은 구현해서 pr 넣어보겠습니다!

seo-rii avatar Jan 12 '21 06:01 seo-rii

추가적인 버그들을 발견했습니다.

  • [x] 보이스리플이 정상적으로 표시되지 않는 문제
  • [x] 추천 버튼을 누를 시 추천수 포맷팅(세자리 단위로 쉼표)가 깨지는 문제
  • [ ] 디시콘 위에 마우스를 올릴 시 움직이는 문제

seo-rii avatar Jan 12 '21 08:01 seo-rii

추가 요청사항입니다.

  • [ ] 디시의 불안정한 서버를 감안하여 preview에서 오류로 인한 요청 실패시(삭제가 아닌) 최대 n번 재시도
  • [ ] image에 margin 주기

seo-rii avatar Jan 12 '21 15:01 seo-rii

@Seo-Rii 미니 갤러리 preview 자체는 잘 동작합니다.

So-chiru avatar Jan 13 '21 04:01 So-chiru

@So-chiru 다시 테스트하니까 잘되네요. 아까 왜안됐지;;;

seo-rii avatar Jan 13 '21 04:01 seo-rii