Skip to content

[#353,#354] 로컬라이징에 더 적합하도록 문자열과 UI을 수정한다#355

Merged
opficdev merged 15 commits intodevelopfrom
feat/#353-l10n
Apr 3, 2026
Merged

[#353,#354] 로컬라이징에 더 적합하도록 문자열과 UI을 수정한다#355
opficdev merged 15 commits intodevelopfrom
feat/#353-l10n

Conversation

@opficdev
Copy link
Copy Markdown
Owner

@opficdev opficdev commented Apr 3, 2026

@opficdev opficdev self-assigned this Apr 3, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

이 풀리퀘스트는 로컬라이제이션 키를 nav_ 접두사 형식으로 표준화하고 한국어 번역 어투를 통일하는 작업을 수행했습니다. 또한 ProfileHeatmapView에서 요일 레이블이 셀 크기에 맞춰 동적으로 조정되도록 개선하고 ProfileTrendChartView의 레이아웃을 정리했습니다. 리뷰에서는 Q%lld 키의 값이 삭제되어 영문 환경에서 분기 표시가 누락될 수 있는 문제를 지적하고 수정을 제안했습니다.

Comment on lines 1390 to 1392
"Q%lld" : {
"localizations" : {
"en" : {
"stringUnit" : {
"state" : "translated",
"value" : "Q%1$lld"
}
}
}
},
"rainbow" : {
"extractionState" : "manual",
"localizations" : {
"en" : {
"stringUnit" : {
"state" : "translated",
"value" : "Rainbow"
}
},
"ko" : {
"stringUnit" : {
"state" : "translated",
"value" : "무지개"
}
}
}

},
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

high

이 키가 비어있어 영어 로컬라이제이션 시 분기(Quarter)가 올바르게 표시되지 않을 수 있습니다. Q%lld 형식으로 값을 다시 추가하고, 번역이 필요 없는 형식 지정자이므로 shouldTranslate: false를 설정하는 것을 권장합니다.

    "Q%lld" : {
      "localizations" : {
        "en" : {
          "stringUnit" : {
            "state" : "translated",
            "value" : "Q%lld"
          }
        }
      },
      "shouldTranslate" : false
    },

@opficdev opficdev merged commit 6eb69cd into develop Apr 3, 2026
1 check passed
@opficdev opficdev deleted the feat/#353-l10n branch April 3, 2026 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant