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

Issue 4912041: New short and long lyric ties. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 8 months ago by Bertrand Bordage
Modified:
12 years, 7 months ago
Reviewers:
pkx166h, janek
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

New short and long lyric ties.

Patch Set 1 #

Patch Set 2 : Long tie removed; default tie resized. #

Patch Set 3 : Remove 'è' in tied-lyric. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+91 lines, -47 lines) Patch
M Documentation/de/notation/vocal.itely View 1 1 chunk +5 lines, -5 lines 0 comments Download
M Documentation/es/notation/vocal.itely View 1 1 chunk +5 lines, -5 lines 0 comments Download
M Documentation/fr/notation/vocal.itely View 1 1 chunk +5 lines, -5 lines 0 comments Download
M Documentation/notation/vocal.itely View 1 1 chunk +5 lines, -5 lines 0 comments Download
M mf/feta-ties.mf View 1 2 chunks +33 lines, -9 lines 0 comments Download
M scm/define-markup-commands.scm View 1 2 1 chunk +38 lines, -18 lines 0 comments Download

Messages

Total messages: 8
Bertrand Bordage
Hi everyone, This follows 8d148ea05fa4b34f8cc3407e112363d715b27ad8 This is fully working, except for a small issue in ...
12 years, 8 months ago (2011-08-17 17:39:40 UTC) #1
janek
Metafont stuff LGTM. I can't really speak for the scheme stuff, but i've looked at ...
12 years, 7 months ago (2011-08-27 05:38:53 UTC) #2
Bertrand Bordage
Thanks Janek. Does someone else thinks the long tie should be removed? Bertrand
12 years, 7 months ago (2011-09-15 18:09:26 UTC) #3
pkx166h
Passes make, a full make doc and reg tests. Output of NR example attached here: ...
12 years, 7 months ago (2011-09-15 20:49:22 UTC) #4
janek
2011/9/15 <pkx166h@gmail.com>: > Passes make, a full make doc and reg tests. > > Output ...
12 years, 7 months ago (2011-09-15 21:19:39 UTC) #5
Bertrand Bordage
New patch set. After thinking about that during three weeks, I decided to remove the ...
12 years, 7 months ago (2011-09-17 22:34:27 UTC) #6
janek
2011/9/18 <bordage.bertrand@gmail.com>: > New patch set. > > After thinking about that during three weeks, ...
12 years, 7 months ago (2011-09-18 09:38:12 UTC) #7
Bertrand Bordage
12 years, 7 months ago (2011-09-19 07:33:58 UTC) #8
Pushed as 2fff263f10fd542454455994aea5ff3bbe075c7d
Sign in to reply to this message.

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