UVCXR/Simulation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ssl/250429 #7

Closed
Ghost wants to merge 0 commits from ssl/250429 into main
pull from: ssl/250429
merge into: UVCXR:main
Conversation 1 Commits - Files Changed - +770 -209
Ghost commented 2025-04-29 11:58:13 +09:00
Copy Link
No description provided.
Ghost added 2 commits 2025-04-29 11:58:13 +09:00
스크립트 변경 테스트 0f8f4bfa6d
Merge pull request '스크립트 변경 테스트' (#4) from ssl/250428 into ssl/250429
All checks were successful
Code Review / code-review (pull_request) Successful in 22s
Details
1ece7ac1a9 
Reviewed-on: http://220.90.135.190:3000/neitt/Simulation/pulls/4
lh commented 2025-04-29 11:58:36 +09:00
Collaborator
Copy Link

Raktbeej Code Review:

Summary:

아래 변경점(디프)은 Unity 씬에서 기존에 사용하던 Label (1) GameObject(UI Image 베이스)의 제거와, 새로 Text (TMP) GameObject(TextMeshProUGUI 기반)의 추가입니다.

이 변경은 일반 Unity UI의 Image 컴포넌트로 만들던 Label을 TextMeshPro 텍스트로 교체한 것으로 분석할 수 있습니다.

상세 비교

제거된 부분

  • GameObject: Label (1)
    • RectTransform (Anchor: {0,0}, Size: 320x30)
    • CanvasRenderer
    • Image 컴포넌트 (m_Script: guid: fe87c0e1cc204ed48ad3b37840f39efc)
      • 색상 a=0 (완전투명)
  • 부모: {fileID: 480905216}

추가된 부분

  • GameObject: Text (TMP)
    • RectTransform (Anchor: {0,1} ~ {1,1}, Size: y=30, y_position = -15)
    • CanvasRenderer
    • TextMeshProUGUI 컴포넌트 (m_Script: guid: f4688fdb7df04437aeb418b961361dc5)
      • 텍스트: "Label"
      • Font: guid: f033b96aa66acfd4b84aa16e32cfc660 (TMP용 FontAsset)
      • 폰트사이즈 24
      • 정렬: 가로 중앙, 세로 상단
      • 부모: {fileID: 955865558}

의도와 주의사항

변경 의도

  • Label(배경)이었던 투명 Image 오브젝트를 삭제.
  • 대신 TextMeshPro의 텍스트 오브젝트로 명확하게 레이블(텍스트)을 표시하도록 변경.

이점

  • TextMeshPro를 사용해 훨씬 읽기 좋고, 커스터마이징 좋은 텍스트 제공.
  • 불필요한 투명 이미지로 UI 계층 낭비를 줄임.
  • 텍스트는 오브젝트 이름이 "Label"로 명확(유지보수, 검색 용이).
  • RectTransform에서 Anchor, 위치, 부모 등이 다르므로 레이아웃에 변동 영향이 있을 수 있음.

주의점/점검 사항

  • 기존 부모(480905216) → 새로운 부모(955865558)로 부모가 변경(부모 오브젝트 관계 체크 필요)
  • TextMeshPro 폰트 에셋(guid: f033b96aa66acfd4b84aa16e32cfc660)이 유효한지, 빌드에 포함되는지 확인 필요
  • 기존 Label을 참조(Find, GetChild 등)하던 코드 있으면 이름, 계층 변경으로 영향 받는지 검토 필요

결론

이 변경은 UI 라벨을 "Image" → "TextMeshProUGUI"로 개선한 것으로, 가독성과 유지보수성이 향상됩니다.
(혹시 기존 Label을 Color나 컨테이너 용도로 사용했다면, 대체 컨테이너가 필요한지 점검 필요합니다.)

요약 문장

이 PR은 기존 Image 기반 Label UI 오브젝트를 삭제하고, 그 자리에 TextMeshPro 기반의 텍스트 오브젝트로 치환하는 변경입니다.
구현상/코드상 기존 라벨 오브젝트-이름, 부모, 참조 관계 영향 가능성이 있으므로 코드 점검 추천합니다.

추가로 궁금한 부분이 있다면 알려주세요!

(예: 계층구조, C# 코드 영향점, 기타 유닛테스트 관점 등)

Raktbeej Code Review: ## Summary: 아래 변경점(디프)은 Unity 씬에서 기존에 사용하던 `Label (1)` GameObject(UI Image 베이스)의 제거와, 새로 `Text (TMP)` GameObject(TextMeshProUGUI 기반)의 추가입니다. 이 변경은 **일반 Unity UI의 Image 컴포넌트로 만들던 Label을 TextMeshPro 텍스트로 교체**한 것으로 분석할 수 있습니다. --- ### 상세 비교 #### 제거된 부분 - GameObject: Label (1) - RectTransform (Anchor: {0,0}, Size: 320x30) - CanvasRenderer - Image 컴포넌트 (`m_Script: guid: fe87c0e1cc204ed48ad3b37840f39efc`) - 색상 a=0 (완전투명) - 부모: {fileID: 480905216} #### 추가된 부분 - GameObject: Text (TMP) - RectTransform (Anchor: {0,1} ~ {1,1}, Size: y=30, y_position = -15) - CanvasRenderer - TextMeshProUGUI 컴포넌트 (`m_Script: guid: f4688fdb7df04437aeb418b961361dc5`) - 텍스트: "Label" - Font: guid: f033b96aa66acfd4b84aa16e32cfc660 (TMP용 FontAsset) - 폰트사이즈 24 - 정렬: 가로 중앙, 세로 상단 - 부모: {fileID: 955865558} --- ## 의도와 주의사항 ### 변경 의도 - Label(배경)이었던 투명 Image 오브젝트를 삭제. - 대신 TextMeshPro의 텍스트 오브젝트로 명확하게 **레이블(텍스트)을 표시**하도록 변경. ### 이점 - **TextMeshPro**를 사용해 훨씬 읽기 좋고, 커스터마이징 좋은 텍스트 제공. - 불필요한 투명 이미지로 UI 계층 낭비를 줄임. - 텍스트는 오브젝트 이름이 `"Label"`로 명확(유지보수, 검색 용이). - RectTransform에서 Anchor, 위치, 부모 등이 다르므로 레이아웃에 변동 영향이 있을 수 있음. ### 주의점/점검 사항 - 기존 부모(480905216) → 새로운 부모(955865558)로 부모가 변경(부모 오브젝트 관계 체크 필요) - TextMeshPro 폰트 에셋(guid: f033b96aa66acfd4b84aa16e32cfc660)이 유효한지, 빌드에 포함되는지 확인 필요 - 기존 Label을 참조(Find, GetChild 등)하던 코드 있으면 이름, 계층 변경으로 영향 받는지 검토 필요 --- ## 결론 이 변경은 **UI 라벨을 "Image" → "TextMeshProUGUI"로 개선**한 것으로, 가독성과 유지보수성이 향상됩니다. (혹시 기존 Label을 Color나 컨테이너 용도로 사용했다면, 대체 컨테이너가 필요한지 점검 필요합니다.) ### 요약 문장 > 이 PR은 기존 Image 기반 Label UI 오브젝트를 삭제하고, 그 자리에 TextMeshPro 기반의 텍스트 오브젝트로 치환하는 변경입니다. > **구현상/코드상 기존 라벨 오브젝트-이름, 부모, 참조 관계 영향 가능성이 있으므로 코드 점검 추천합니다.** --- ### 추가로 궁금한 부분이 있다면 알려주세요! (예: 계층구조, C# 코드 영향점, 기타 유닛테스트 관점 등)
Ghost closed this pull request 2025-04-30 12:21:41 +09:00
All checks were successful
Code Review / code-review (pull_request) Successful in 22s
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
khi khm6376 lh lwj njh UVCXR
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: UVCXR/Simulation#7
Delete Branch "ssl/250429"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?