Conversation
- 제보하기 제목 글자 수 제한 - 제보 히스토리에서 제보하기 제목을 최대 2줄로 표시하도록 수정
Walkthrough이 변경 사항은 보고서 UI의 텍스트 처리를 개선합니다. 제목 라벨을 여러 줄로 지원하도록 조정하고, 문자 수 제한 기능을 추가하며, 레이아웃 제약의 우선순위를 조정합니다. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. 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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
Projects/Presentation/Sources/Report/View/ReportRegistrationViewController.swift (1)
325-328:⚠️ Potential issue | 🟡 Minor
locationTitleLabel이contentTextCountLabel아래가 아닌reportContentTextView아래에 고정되어 있습니다.
contentTextCountLabel이reportContentTextView아래에 배치되어 있으므로,locationTitleLabel의 상단 제약은contentTextCountLabel.snp.bottom을 기준으로 설정해야 레이아웃 체인이 올바르게 연결됩니다. 현재 상태에서는contentTextCountLabel과locationTitleLabel사이의 간격이 4pt밖에 되지 않아 시각적으로 겹칠 수 있습니다.🔧 수정 제안
locationTitleLabel.snp.makeConstraints { make in make.top - .equalTo(reportContentTextView.snp.bottom) + .equalTo(contentTextCountLabel.snp.bottom) .offset(Layout.titleLabelTopSpacing)
🤖 Fix all issues with AI agents
In `@Projects/Presentation/Sources/Report/View/ReportDetailViewController.swift`:
- Line 155: Remove the incorrect numberOfLines assignment on titleLabel and
instead set the content label to allow wrapping: delete the line setting
titleLabel.numberOfLines = 2, and in the .title section use the content label
(titleContentLabel or the local descriptionLabel used for the .title case) to
set numberOfLines = 2 so long titles wrap across two lines.
🧹 Nitpick comments (2)
Projects/Presentation/Sources/Report/View/ReportRegistrationViewController.swift (1)
65-68: 쉼표 위치가 비정상적입니다.
maxLength매개변수 앞의 쉼표가 새 줄의 시작에 위치해 있어 가독성이 떨어집니다.♻️ 수정 제안
private let reportTitleTextView = ReportTextView( type: .editable, - placeholder: "제보 제목을 작성해주세요." - ,maxLength: 50) + placeholder: "제보 제목을 작성해주세요.", + maxLength: 50 + )Projects/Presentation/Sources/Report/View/Component/ReportHistory/ReportHistoryTableViewCell.swift (1)
17-17:titleLabelMaxHeight가 사용되지 않고 있습니다.
Layout.titleLabelMaxHeight가 정의되어 있으나 실제 제약 조건에서 참조되지 않습니다. 사용하지 않는 상수라면 제거하거나, 필요하다면titleLabel에 최대 높이 제약으로 적용하는 것을 고려해 주세요.
| titleLabel.text = reportDetailContentType.title | ||
| titleLabel.font = BitnagilFont(style: .body2, weight: .semiBold).font | ||
| titleLabel.textColor = BitnagilColor.gray10 | ||
| titleLabel.numberOfLines = 2 |
There was a problem hiding this comment.
numberOfLines = 2가 잘못된 레이블에 설정되어 있습니다.
titleLabel은 섹션 헤더("제목", "카테고리" 등)를 표시하는 로컬 변수입니다. 제보 제목 내용을 표시하는 titleContentLabel에 numberOfLines = 2를 설정해야 합니다.
현재 titleContentLabel은 기본값 numberOfLines = 1이므로, 긴 제목이 잘릴 수 있습니다.
🐛 수정 제안
Line 155에서 titleLabel.numberOfLines = 2를 제거하고, .title 케이스의 descriptionLabel에 설정:
- titleLabel.numberOfLines = 2
var descriptionLabel: UILabel
switch reportDetailContentType {
case .title:
descriptionLabel = titleContentLabel
+ descriptionLabel.numberOfLines = 2
case .category:🤖 Prompt for AI Agents
In `@Projects/Presentation/Sources/Report/View/ReportDetailViewController.swift`
at line 155, Remove the incorrect numberOfLines assignment on titleLabel and
instead set the content label to allow wrapping: delete the line setting
titleLabel.numberOfLines = 2, and in the .title section use the content label
(titleContentLabel or the local descriptionLabel used for the .title case) to
set numberOfLines = 2 so long titles wrap across two lines.
1 participant
🌁 Background
👩💻 Contents
Summary by CodeRabbit
새로운 기능
개선 사항