[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-i18n-doc
Subject: [Differential] [Request, 4 lines] D4356: Fix UI string: remove placeholders for arguments
From: Luigi Toscano <noreply () phabricator ! kde ! org>
Date: 2017-01-31 0:50:50
Message-ID: differential-rev-PHID-DREV-deydfxb5s2dmn3aaahqx-req () phabricator ! kde ! org
[Download RAW message or body]
ltoscano created this revision.
ltoscano added a reviewer: chehrlic.
ltoscano added a subscriber: Localization.
REVISION SUMMARY
%1, %2,... are interpreted as placeholders for arguments leading to
an I18n_ARGUMENT_MISSING error in the visible string. Rewrite them
as more generic placeholder which are not replaced, as it was done
for other strings in the same file.
TEST PLAN
When accessing the configuration of 'Diff & Merge', "What's this"
on "External diff display", there is no more I18n_ARGUMENT_MISSING
error.
REPOSITORY
R220 Subversion Client (KDE SVN)
BRANCH
fix-i18n-error
REVISION DETAIL
https://phabricator.kde.org/D4356
AFFECTED FILES
src/settings/diffmergesettings.ui
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: ltoscano, chehrlic
Cc: #localization
[Attachment #3 (text/html)]
<table><tr><td style="">ltoscano created this revision.<br />ltoscano added a \
reviewer: chehrlic.<br />ltoscano added a subscriber: Localization. </td><a \
style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; \
color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; \
background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; \
border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D4356" \
rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION \
SUMMARY</strong><div><p>%1, %2,... are interpreted as placeholders for arguments \
leading to<br /> an I18n_ARGUMENT_MISSING error in the visible string. Rewrite \
them<br /> as more generic placeholder which are not replaced, as it was done<br />
for other strings in the same file.</p></div></div><br /><div><strong>TEST \
PLAN</strong><div><p>When accessing the configuration of 'Diff & \
Merge', "What's this"<br /> on "External diff display", \
there is no more I18n_ARGUMENT_MISSING<br /> error.</p></div></div><br \
/><div><strong>REPOSITORY</strong><div><div>R220 Subversion Client (KDE \
SVN)</div></div></div><br \
/><div><strong>BRANCH</strong><div><div>fix-i18n-error</div></div></div><br \
/><div><strong>REVISION DETAIL</strong><div><a \
href="https://phabricator.kde.org/D4356" \
rel="noreferrer">https://phabricator.kde.org/D4356</a></div></div><br \
/><div><strong>AFFECTED \
FILES</strong><div><div>src/settings/diffmergesettings.ui</div></div></div><br \
/><div><strong>EMAIL PREFERENCES</strong><div><a \
href="https://phabricator.kde.org/settings/panel/emailpreferences/" \
rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br \
/><div><strong>To: </strong>ltoscano, chehrlic<br /><strong>Cc: \
</strong>Localization<br /></div>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic