[fix] 이미지 미로드 시 early return

[seongwon030]

#️⃣연관된 이슈

ex) #이슈번호, #이슈번호

📝작업 내용

• 이미지 미로드 시에도 MAIN_POPUP_NOT_SHOWN 이벤트가 찍힘.
• 미로드 시 얼리리턴하도록 변경

중점적으로 리뷰받고 싶은 부분(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

논의하고 싶은 부분(선택)

논의하고 싶은 부분이 있다면 작성해주세요.

🫡 참고사항

Summary by CodeRabbit

버그 수정

• 팝업 성능 개선 - 이미지 로드 전 불필요한 처리를 건너뛰도록 최적화하여 초기 렌더링 시 팝업 창의 반응성이 향상되었습니다.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
moadong	Ready	Preview, Comment	Feb 11, 2026 8:07am

[coderabbitai]

Warning

.coderabbit.yaml has a parsing error

The CodeRabbit configuration file in this repository has a parsing error and default settings were used instead. Please fix the error(s) in the configuration file. You can initialize chat with CodeRabbit to get help with the configuration file.

💥 Parsing errors (1)

Validation error: Invalid regex pattern for base branch. Received: "**" at "reviews.auto_review.base_branches[0]"

⚙️ Configuration instructions

• Please see the configuration documentation for more information.
• You can also validate your configuration using the online YAML validator.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Walkthrough

팝업 가시성 이펙트에 이미지 로드 상태 확인 로직을 추가하여 imageLoaded가 false일 때 불필요한 연산을 스킵하는 초기 반환 추가.

Changes

Cohort / File(s)	Summary
팝업 성능 최적화 frontend/src/pages/MainPage/components/Popup/Popup.tsx	imageLoaded 상태 확인 초기 반환 추가로 이미지 미로드 시 불필요한 isHidden 체크 및 로직 실행 방지

예상 코드 리뷰 노력

🎯 1 (Trivial) | ⏱️ ~3 minutes

제안된 검토자

• suhyun113
• lepitaaar

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 inconclusive)

Check name	Status	Explanation	Resolution
Linked Issues check	❓ Inconclusive	PR은 이미지 미로드 시 불필요한 이벤트 기록을 방지하는 버그 수정을 포함하고 있으나, 링크된 이슈 MOA-634의 AB테스트 팝업 제거 작업과의 직접적인 연관성이 명확하지 않습니다.	이미지 미로드 버그 수정이 AB테스트 팝업 제거 작업의 필수 요구사항인지, 아니면 독립적인 버그 수정인지 명확히 하기 위해 이슈와의 관계를 설명해주시기 바랍니다.
Out of Scope Changes check	❓ Inconclusive	변경 사항은 Popup 컴포넌트의 이미지 로드 관련 로직 개선에만 집중하고 있으며, AB테스트 팝업 제거 작업 범위와의 관계가 불명확합니다.	변경 사항이 MOA-634의 팝업 제거 작업에 포함되는지 확인하고, 필요시 이슈 설명을 업데이트하여 범위를 명확히 하시기 바랍니다.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 변경 사항의 주요 내용을 명확하게 반영하고 있습니다. 이미지 미로드 시 얼리 리턴을 추가하는 코드 변경을 간결하게 설명하고 있습니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chore/#1178-cleanup-ab-test-popup-MOA-634

---

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

frontend/src/pages/MainPage/components/Popup/Popup.tsx (1)

42-42: imageLoaded 조건이 이제 중복됩니다.

Line 38에서 !imageLoaded일 때 이미 리턴하므로, 이 지점에 도달하면 imageLoaded는 항상 true입니다. 조건에서 imageLoaded를 제거하면 가독성이 향상됩니다.

♻️ 중복 조건 제거 제안

-    if (isMobile && !isHidden && imageLoaded) {
+    if (isMobile && !isHidden) {

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}