Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29336165: No issue - Non breakable space fixes in localized strings (Closed)

Created:
Feb. 9, 2016, 12:26 p.m. by saroyanm
Modified:
Feb. 9, 2016, 1:11 p.m.
Reviewers:
tamara
Visibility:
Public.

Description

No issue - Non breakable space fixes in localized strings

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+73 lines, -73 lines) Patch
M locales/de/about.json View 3 chunks +5 lines, -5 lines 0 comments Download
M locales/de/acceptable-ads.json View 8 chunks +8 lines, -8 lines 0 comments Download
M locales/es/about.json View 1 2 chunks +2 lines, -2 lines 0 comments Download
M locales/es/acceptable-ads.json View 11 chunks +11 lines, -11 lines 0 comments Download
M locales/fr/about.json View 4 chunks +6 lines, -6 lines 0 comments Download
M locales/fr/acceptable-ads.json View 24 chunks +41 lines, -41 lines 0 comments Download

Messages

Total messages: 4
saroyanm
I have noticed some changes in Spanish translation of about page Should I keep that ...
Feb. 9, 2016, 12:30 p.m. (2016-02-09 12:30:42 UTC) #1
tamara
On 2016/02/09 12:30:42, saroyanm wrote: > I have noticed some changes in Spanish translation of ...
Feb. 9, 2016, 12:44 p.m. (2016-02-09 12:44:42 UTC) #2
tamara
On 2016/02/09 12:44:42, tamara wrote: > On 2016/02/09 12:30:42, saroyanm wrote: > > I have ...
Feb. 9, 2016, 12:44 p.m. (2016-02-09 12:44:50 UTC) #3
saroyanm
Feb. 9, 2016, 1:09 p.m. (2016-02-09 13:09:40 UTC) #4
On 2016/02/09 12:44:50, tamara wrote:
> On 2016/02/09 12:44:42, tamara wrote:
> > On 2016/02/09 12:30:42, saroyanm wrote:
> > > I have noticed some changes in Spanish translation of about page Should I
> keep
> > > that changes ?
> > 
> > As already mentioned, only keep the non-breaking spaces. Thanks!
> 
> LGTM

Done, I also kept the <strong> tag change as you requested

Powered by Google App Engine
This is Rietveld