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 1 commit into from
Oct 20, 2024
Merged

코드블럭 복사 기능 추가 #22

merged 1 commit into from
Oct 20, 2024

Conversation

youngminss
Copy link
Owner

작업 내용

  • 코드블럭 복사 기능 추가

메모

  • 코드블록 구현을 위해 rehype-pretty-code plugin 을 사용 중이었고, experimental 기능이긴하나 copy button 옵션이 있어서 사용해보려 했으나, rsc 에서 적용 시 error 이슈가 있으므로 기각
  • 복사 버튼이 붙은 커스텀 pre component 를 pre element 대신 사용하도록 수정 (cc. Error: Expected onClick listener to be a function, instead got a value of string type. rehype-pretty/rehype-pretty-code#235 (comment))
  • 위 cc 를 참고해서 기능 구현은 되었으나, 스타일 측면으로 line 이 길어져서 스크롤이 되는 형태일 때 copy 버튼이 항상 우상단에 위치해야하는 요구사항을 만족시키지 못함, 최종적으로 pre element 와 copy button 을 감싸하는 div 컨테이너를 wrapping 함
    • copy button 은 div 기준으로 우상단을 계산하므로 항상 우상단에 위치하고
    • 스크롤이 될 수 있는 pre 코드블록은 코드블록대로 개별적으로 스크롤링 되도록

체크리스트

  • chrome
  • firefox
  • safari

@youngminss youngminss added the feature 기능 작업 label Oct 20, 2024
@youngminss youngminss self-assigned this Oct 20, 2024
Copy link

vercel bot commented Oct 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
youngminss-log ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 20, 2024 9:16am

@youngminss youngminss merged commit d671757 into main Oct 20, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 기능 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant