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

에디터 관련 수정 #22

Merged
merged 7 commits into from
Aug 5, 2024
Merged

에디터 관련 수정 #22

merged 7 commits into from
Aug 5, 2024

Conversation

xxxjinn
Copy link
Collaborator

@xxxjinn xxxjinn commented Aug 3, 2024

close #19
(title, 에디터에 내용 있어야지 버튼 활성화되는 로직 구현)

  • ckeditor import 에러나서 또 헤매다가.. 하 그냥 react-quill (원래 있던거) 사용해봄
    • toolbar 관련 이슈 해결법
      • react-quill은 toolbar를 모듈로 만들어서 커스텀이 가능한데, 이 커스텀한 toolbar를 사용하는 게 렌더링에서 문제가 되었음. 그래서 모듈을 따로 사용하지 않고 직접 modules라는 변수를 정의해 toolbar 커스텀함
    • 브라우저 경고 (react-quill 라이브러리 자체의 이슈)
      • [Deprecation] Listener added for a 'DOMNodeInserted' mutation event. This event type is deprecated, and will be removed from this browser VERY soon. Usage of this event listener will cause performance issues today, and represents a large risk of imminent site breakage. Consider using MutationObserver instead. See https://chromestatus.com/feature/5083947249172480 for more information. => 브라우저에서 더 이상 사용하지 않는 메서드를 내부적으로 사용하고 있어서 뜸
      • 관련해 이슈를 계속 찾아보니까 어떤 사람이 자기가 포크해서 그 에러 해결해놨다고 그 라이브러리 쓰라고 함. (react-quill-new) https://github.com/VaguelySerious/react-quill
      • 경고 더 이상 안뜸 => 해결 완료

새로운 문제

(글 목록) bullet이 적용 안되고 숫자만 적용됨

  • << 이게 안되고 1. << 이것만 되는 중

해당 레포 pr에 해결법인 것 같은게 올라와져 있어서 그대로 라이브러리 코드 내부적으로 똑같이 수정했으나 해결되지 않음 일단 이 상태로 머지함

KakaoTalk_20240805_191338804.mp4

@xxxjinn xxxjinn self-assigned this Aug 3, 2024
@xxxjinn xxxjinn marked this pull request as ready for review August 5, 2024 10:05
@xxxjinn xxxjinn merged commit 3e8035e into development Aug 5, 2024
@xxxjinn xxxjinn deleted the editor/refactor branch August 5, 2024 10:07
@xxxjinn xxxjinn restored the editor/refactor branch August 5, 2024 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

각 게시판 업로드 페이지 (공지사항, 행사, FAQ) 로직 구현
1 participant