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

[#366] Fix layout shifting on Viewer #367

Merged
merged 3 commits into from
May 11, 2024
Merged

Conversation

2wheeh
Copy link
Collaborator

@2wheeh 2wheeh commented May 10, 2024

resolve #366

Changes

  1. upstream 패키지인 lexical 에서 html 을 만들어내기 위해 사용하는 paragraphNode의 exportDOM 메서드에 버그가 존재합니다.

    주입된 theme 의 indent 값을 참조하지않고 하드코딩된 20px 을 사용하도록 구현되어있습니다. 따라서 lexical에서 이슈를 해결해주기 전까지 theme의 indent 값을 사용하도록 paragraphNode 의 html export 설정을 오버라이드 합니다.

  2. 플레이스홀더의 html과 스타일을 일부 수정하여 레이아웃 쉬프팅을 방지합니다.

Before

2024-05-11.8.45.57.mov

After

2024-05-11.8.41.34.mov

@2wheeh 2wheeh added bug Something isn't working ui Something about UI labels May 10, 2024
@2wheeh 2wheeh requested review from isutare412 and skgndi12 May 10, 2024 23:53
@2wheeh 2wheeh self-assigned this May 10, 2024
Copy link

@2wheeh 2wheeh force-pushed the fix/issue-366/shifting branch from a4487db to 24ffb84 Compare May 11, 2024 02:24
Copy link

Copy link
Collaborator

@isutare412 isutare412 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 👍

@2wheeh 2wheeh merged commit c175385 into develop May 11, 2024
4 checks passed
@2wheeh 2wheeh deleted the fix/issue-366/shifting branch May 11, 2024 03:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ui Something about UI
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Remove shifting on review detail
2 participants