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

Handle element that has contenteditable as false #7

Open
sohee-lee7 opened this issue Jun 18, 2019 · 1 comment
Open

Handle element that has contenteditable as false #7

sohee-lee7 opened this issue Jun 18, 2019 · 1 comment

Comments

@sohee-lee7
Copy link
Owner

sohee-lee7 commented Jun 18, 2019

구분선 주변에서 좌우 키로 이동 시 커서(range)가 <hr> 테그 양옆에 위치하게 되고 (실제 랜더링은 css의 영향으로 <hr> 바로 아래에 렌더링 된다.) 그 순간 텍스트를 입력하면 구분선 위아래에 텍스트가 입력된다.

이를 수정하기 위해 구분선을 아래와 같은 형태의 돔으로 운영하기로 하였다. (nhn/tui.editor#543)

<div contenteditable="false">
  <hr contenteditable="false">
</div>

이때의 스콰이어의 문제는 다른 텍스트를 나타내는 <div><div contenteditable="false">로 머지될 소지가 있으며, delete키와 backspace 키로 구분선을 지울 때 <div contenteditable="false"> 안에 있는 <hr contenteditable="false"> 만 지운다.

@sohee-lee7
Copy link
Owner Author

  • <div><div contenteditable="false">로 머지되는 문제
    • areAlike 에서 노드가 같은지 확인 할 때 contenteditable도 확인하도록 개선
  • delete키와 backspace 키로 구분선을 지울 때 <hr contenteditable="false"> 만 지운다.
    • delete키와 backspace 키 동작시 contenteditablefalse인 것을 지울 때 부모들도 확인하여 지우도록 개선

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

No branches or pull requests

1 participant