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

Feature/#011 순서있는 리스트 숫자 구현 #214

Merged
merged 7 commits into from
Nov 27, 2024

Conversation

Ludovico7
Copy link
Collaborator

📝 변경 사항

🔍 변경 사항 설명

  • 순서있는 리스트 생성시 순서를 반영해서 iconBlock의 숫자를 갱신합니다.
    • 리스트 생성시, 이전 인덱스보다 1 증가시켜서 생성
    • 리스트 중간에서 기본 블록으로 변경할 때, 두 리스트로 분할해서 각각 1부터 시작
    • indent 증가시 1부터 시작
    • indent 감소시 기존 순서에 맞게 변경
    • 두개의 순서 리스트 병합시, 먼저 있는 리스트에 합쳐서 숫자 시작

CRDT의 BlockLinkedList에 updateAllOrderedListIndices 메서드를 구현해서 현재 연산이 순서 리스트일 경우, 모든 블록을 순회하며 리스트의 index를 갱신합니다. 기존에는 각 케이스별로 코드를 작성했는데 이렇게 하다보니까 에러도 많이 발생하고 코드의 길이가 늘어나서 LinkedList 메서드로 분리해서 사용하는 식으로 구현했습니다.
블록을 모두 순회하는 로직이다보니 자주 실행하면 성능에 영향을 줄 수 있을것 같아 순서리스트 연산일때만 실행시키도록 분기 처리했습니다.

🙏 질문 사항

  • 혹시 엣지케이스나 에러케이스 등 문제가 발생하는 부분이 있으면 피드백 부탁드립니다!

📷 스크린샷 (선택)

orderedlist.mov

✅ 작성자 체크리스트

  • Self-review: 코드가 스스로 검토됨
  • Unit tests 추가 또는 수정
  • 로컬에서 모든 기능이 정상 작동함
  • 린터 및 포맷터로 코드 정리됨
  • 의존성 업데이트 확인
  • 문서 업데이트 또는 주석 추가 (필요 시)

@Ludovico7 Ludovico7 added FE 프론트엔드 작업 Feat 새로운 기능 추가나 기존 기능 확장 작업 P1 긴급/즉시 반영 labels Nov 26, 2024
@Ludovico7 Ludovico7 self-assigned this Nov 26, 2024
Copy link
Collaborator

@minjungw00 minjungw00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고많으셨습니다!

updateEditorState();
if (prevBlock.type === "ol" && nextBlock?.type === "ol") {
editorCRDT.LinkedList.updateAllOrderedListIndices();
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이전과 이후가 "ol"이여야 실행되게 수정됐군요! 확인했습니다.

Copy link
Collaborator

@hyonun321 hyonun321 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다 !!

@github-actions github-actions bot merged commit 11b6650 into dev Nov 27, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 작업 Feat 새로운 기능 추가나 기존 기능 확장 작업 P1 긴급/즉시 반영
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants