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/#81 PR 수정 #89

Merged
merged 3 commits into from
Jul 24, 2023
Merged

Feature/#81 PR 수정 #89

merged 3 commits into from
Jul 24, 2023

Conversation

ChoiWonBeen
Copy link
Member

[#81] PR에 대한 수정

@hyejun0228 위지윅 에디터가 깨지는 이유를 발견하는 게 힘들 것 같아서 PR로 정리해서 올려요. (이전 코드 작성자가 쓴 부분들도 있고 해서..)
어떤 점들이 변경되어야하는지 보고 파악 후 병합하면 좋을 것 같아요.

  • 적절하지 못한 <span> 태그에 의해 width를 조절하기 힘들었을거예요. 해당 부분을 div로 바꾸니 대부분의 문제가 해결됐습니다.
    • PC, 모바일 환경에서 width값이 특정 값으로 고정이 아닌, **100%**로 잘 동작합니다 ㅋㅋ
  • className 컨벤션을 통일했습니다.
  • 불필요한 고정 height값들을 없앴습니다.
  • 이미지 추가 버튼을 fixed로 수정했습니다.

스타일을 정확하게 확인한 건 아니라, 병합하고난 뒤 디자인과 맞지 않는 부분 있으면 수정해 쓰면 될거예요.

Please check if the PR fulfills these requirements

  • It's submitted to develop branch, not the main branch
  • The commit message follows our guidelines
  • There are no warning message when you run yarn lint
  • Docs updated for breaking changes

Screenshot

image image

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.

2 participants