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

[Release] 리뷰미 v2.0.2 배포 #929

Merged
merged 3 commits into from
Oct 24, 2024
Merged

[Release] 리뷰미 v2.0.2 배포 #929

merged 3 commits into from
Oct 24, 2024

Conversation

BadaHertz52
Copy link
Contributor


🚀 어떤 기능을 구현했나요 ?

  • 형광펜 버그 수정
  • ios 형광펜 안됨 및 위치 버그 해결
  • 형광펜 색상 변경
  • 로고 스크린 리더기 오류 수정
  • URL 모달 뒤로 가기 막기
  • 차트 내림차순 정렬

🔥 어떻게 해결했나요 ?

📝 어떤 부분에 집중해서 리뷰해야 할까요?

📚 참고 자료, 할 말

BadaHertz52 and others added 2 commits October 24, 2024 14:50
* chore: 필요 없는 훅 파일 삭제

* fix: 터치가 아닐 때만, 형광펜 메뉴를 여는 mouseup 이벤트 실행

* fix: selectionInfo를 ref로 저장해 사용하는 방법으로 수정

- 같은 답변에 대한 추가,삭제는 잘 안되지만 여러 답변에 대한 추가,삭제는 됨

* fix: ios에서 findSelectionInfo 반환값인 selection을 파라미터로 받을 때 초기화되는 오류 수정

- 형광펜 추가/삭제 계산에 필요한 값들을 분리해서 반환하는 것으로 수정

* chore: 디버깅용 코드 삭제

* chore: 불필요한 코드 삭제

* design: 주관식 답변이 짧을 때, 툴팁이 화면에 가려지는 오류 수정

* chore: 머지과정에서 복구된 HighlightToggleButtonContainer 파일 삭제

* chore: 리뷰 모아보기 형광펜 관련 목 데이터 변경

* fix: 아이폰, 아이패드 글자 선택 방식에 따른 형광펜 버튼 위치 오류 수정

* design: 형광펜 색 변경

* feat: 로고에 대한 스크린 리더기 사용자 편의성 개선
* feat: URL 모달이 떠있는 동안 뒤로가기, 새로고침 방지

* fix: 뒤로가기 확인 시 모달만 닫히는 문제 수정
…결을 위한 메타 데이터 수정 (#926)

* chore: svg 이미지 사이즈 조정

* fix: merge 과정에서 없어진 투표 내림차순 코드 복원
@BadaHertz52 BadaHertz52 changed the title [All] 리뷰미 v2.0.2 [All] release : 리뷰미 v2.0.2 Oct 24, 2024
@donghoony donghoony changed the title [All] release : 리뷰미 v2.0.2 [Release] 리뷰미 v2.0.2 배포 Oct 24, 2024
Copy link
Contributor

@donghoony donghoony left a comment

Choose a reason for hiding this comment

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

짜라빠빠~ 그대는 아름다워~ 짜 짜라 짜라 빠빠빠~

Copy link
Contributor

@ImxYJL ImxYJL left a comment

Choose a reason for hiding this comment

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

✌✌✌

Copy link
Contributor

@skylar1220 skylar1220 left a comment

Choose a reason for hiding this comment

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

진짜 찐찐 최종 어프루브 내일 화이팅!!!

Copy link
Contributor

@nayonsoso nayonsoso left a comment

Choose a reason for hiding this comment

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

천재천재~~ 머지 커밋 돈 폴겟 ~

Copy link
Contributor

@Kimprodp Kimprodp left a comment

Choose a reason for hiding this comment

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

LeonardoDicaprioClappingGIF

@BadaHertz52 BadaHertz52 merged commit 726955d into release Oct 24, 2024
0 of 5 checks passed
@donghoony donghoony mentioned this pull request Oct 24, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

8 participants