Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(769)

Issue 193890043: Edit Kievan documentation (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 4 months ago by aleksandr.andreev
Modified:
9 years, 3 months ago
Reviewers:
pkx166h, pkx, Jean-Charles
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Edit Kievan documentation Fixing some problems in French and German documentation of Kievan notation.

Patch Set 1 #

Total comments: 1

Patch Set 2 : Fix copy/paste error with German doc #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -8 lines) Patch
M Documentation/de/notation/ancient.itely View 1 4 chunks +7 lines, -7 lines 0 comments Download
M Documentation/fr/notation/ancient.itely View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 8
aleksandr.andreev
Fixing some issues with the French and German documentation of Kievan notation.
9 years, 4 months ago (2015-01-03 12:45:08 UTC) #1
Jean-Charles
LGTM for French https://codereview.appspot.com/193890043/diff/1/Documentation/de/notation/ancient.itely File Documentation/de/notation/ancient.itely (right): https://codereview.appspot.com/193890043/diff/1/Documentation/de/notation/ancient.itely#newcode2670 Documentation/de/notation/ancient.itely:2670: \clef "kievan-do" \kievanOn
9 years, 4 months ago (2015-01-03 14:01:22 UTC) #2
pkx166h
1. I cannot find a tracker for this issue (is that important for translations?). 2 ...
9 years, 4 months ago (2015-01-04 12:13:31 UTC) #3
aleksandr.andreev
Fix copy/paste error with German doc
9 years, 4 months ago (2015-01-04 13:12:43 UTC) #4
aleksandr.andreev
On 2015/01/04 12:13:31, J_lowe wrote: > 1. I cannot find a tracker for this issue ...
9 years, 4 months ago (2015-01-04 13:17:15 UTC) #5
pkx166h
On 2015/01/04 13:17:15, aleksandr.andreev wrote: > On 2015/01/04 12:13:31, J_lowe wrote: > > 1. I ...
9 years, 4 months ago (2015-01-04 13:26:19 UTC) #6
aleksandr.andreev
On 2015/01/04 13:26:19, J_lowe wrote: > On 2015/01/04 13:17:15, aleksandr.andreev wrote: > > On 2015/01/04 ...
9 years, 3 months ago (2015-01-12 11:06:50 UTC) #7
pkx
9 years, 3 months ago (2015-01-12 11:11:38 UTC) #8
On 12/01/15 11:06, aleksandr.andreev@gmail.com wrote:
> On 2015/01/04 13:26:19, J_lowe wrote:
>> On 2015/01/04 13:17:15, aleksandr.andreev wrote:
>> > On 2015/01/04 12:13:31, J_lowe wrote:
>> > > 1. I cannot find a tracker for this issue (is that important for
>> > translations?).
>> > >
>> >
>> > The tracker is here:
>> > https://code.google.com/p/lilypond/issues/detail?id=4241
>> >
>
> Could someone please push this for me, as I do not have push access?
> Thank you.
>
> https://codereview.appspot.com/193890043/
Could you give me a git-formatted patch based on current master and I 
can push it for you.

James
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b