[KJots] Translate statusbar messages
ClosedPublic

Authored by poboiko on Apr 21 2020, 11:43 AM.

Details

Summary

Use i18nc to translate the statusbar messages showing link information

Test Plan

It compiles, the information is still shown

Diff Detail

Repository
R573 KJots
Branch
translate-statusbar (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 25594
Build 25612: arc lint + arc unit
poboiko created this revision.Apr 21 2020, 11:43 AM
Restricted Application added a project: KDE PIM. · View Herald TranscriptApr 21 2020, 11:43 AM
poboiko requested review of this revision.Apr 21 2020, 11:43 AM
pino added a comment.Apr 21 2020, 11:48 AM

The actual i18n changes LGTM, thanks!

src/KJotsMain.cpp
30

no need for the prefix, <KLocalizedString> or <klocalizedstring.h> is fine

poboiko updated this revision to Diff 80757.Apr 21 2020, 12:20 PM

Remove include prefix

pino added a comment.Apr 21 2020, 12:38 PM

LGTM for me from an i18n POV, I'll let @dvratil give a final review/approval.

Thanks!

dvratil accepted this revision.Apr 21 2020, 2:23 PM
This revision is now accepted and ready to land.Apr 21 2020, 2:23 PM
poboiko closed this revision.Apr 21 2020, 4:25 PM