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

[FIX] 검색 결과 포커스 해제 #739

Merged
merged 7 commits into from
Dec 11, 2024
Merged

[FIX] 검색 결과 포커스 해제 #739

merged 7 commits into from
Dec 11, 2024

Conversation

obvoso
Copy link
Collaborator

@obvoso obvoso commented Dec 11, 2024

What is this PR? 🔍

  • 기능 : 이슈에 언급된 검색모달에서 넘어간 페이지의 픽 하이라이트를 클릭 이벤트 발생시 사라지게 했습니다.
  • 이외에 @kimminkyeu 님이 말씀해주신 검색 다이얼로그 내 스타일을 수정했습니다.
  • issue : [bug] 검색 결과 포커스 해제 안됨 #728

ScreenShot 📷

이번 PR에서의 변경점

Before After
Before Image After Image

Precaution

- 수정 전: 평생
- 수정 후: 페이지 내 클릭이벤트 발생 전까지
- 이미지의 경로가 잘못되었을 경우,
예) url: "이미지 경로가 들어갑니다."
등 http로 시작하지 않으면 기본 이미지로 보여줌
- 검색어 입력시 디바운스(유지)되는 이벤트 이외에 엔터, 마우스로 아이콘 클릭시에도 호출되도록 추가
@obvoso obvoso added design 🎨 디자인 관련 frontend 프론트 관련 fix 🔥 bug 수정을 포함한 변경 사항 labels Dec 11, 2024
@obvoso obvoso requested a review from kimminkyeu December 11, 2024 06:41
@obvoso obvoso self-assigned this Dec 11, 2024
Copy link
Collaborator

@dmdgpdi dmdgpdi left a comment

Choose a reason for hiding this comment

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

LGTM

@obvoso obvoso merged commit bcb19d2 into frontend Dec 11, 2024
1 check passed
@Gyaak Gyaak deleted the front/fix/#728 branch December 12, 2024 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design 🎨 디자인 관련 fix 🔥 bug 수정을 포함한 변경 사항 frontend 프론트 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants