[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 &#039;Diff &amp; \
Merge&#039;, &quot;What&#039;s this&quot;<br /> on &quot;External diff display&quot;, \
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