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

v0.5.1 피드백 반영 작업 #322

Merged
merged 3 commits into from
Dec 15, 2022
Merged

v0.5.1 피드백 반영 작업 #322

merged 3 commits into from
Dec 15, 2022

Conversation

doputer
Copy link
Member

@doputer doputer commented Dec 15, 2022

개요

v0.5.1 배포 후 피드백 주신 부분을 수정했습니다.

변경 사항

  • Content 컴포넌트 스타일을 수정했습니다.
    • 이미지 크기가 고정되어 있어서 부모 요소를 뚫고 나오는 오류를 수정했습니다.
    • 글자 줄바꿈이 일어나지 않아서 부모 요소를 뚫고 나오는 오류를 수정했습니다.
  • 검색어 강조 로직 관련 함수 개선했습니다.
    • 영문자 검색어 검색 시 무조건 소문자로 변경되어서 강조되는 오류를 수정했습니다.
    • 최초 등장한 검색어로부터 200자를 자르는 로직을 수정했습니다. 재귀 시 계산하는 것이 아니라 재귀 함수 진입 전에 계산하도록 변경했습니다.
  • TOC에서 같은 이름의 헤딩이 있는 경우 링크가 동작하지 않는 오류를 수정했습니다.

관련 이슈

@doputer doputer added BUG Bug FE Frontend IMPROVE Improve Code labels Dec 15, 2022
@doputer doputer self-assigned this Dec 15, 2022
@github-actions
Copy link

⚡️ Lighthouse report!

Category Score
🟠 Performance 83
🟢 Accessibility 97
🟢 Best Practices 92
🟢 SEO 100

Copy link
Collaborator

@dahyeon405 dahyeon405 left a comment

Choose a reason for hiding this comment

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

마지막 작업하시느라 고생하셨습니다!! 👍 알고리즘 개선하신 건 낼 발표 내용에도 반영해야겠군요 ㅎㅎ

@doputer doputer merged commit 111a9f1 into develop Dec 15, 2022
@doputer doputer deleted the feature/feedback branch December 15, 2022 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BUG Bug FE Frontend IMPROVE Improve Code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants