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

Issue 285650043: blah (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years, 2 months ago by pkx166h
Modified:
8 years, 2 months ago
Reviewers:
Visibility:
Public.

Description

Musicxml: Fix musicxml.py for Reg Test 42a MusicXML regression test 42a-MultiVoice-TwoVoicesOnStaff-Lyrics.xml is missing the first note in voice 2. Subsequently the lyrics of voice 2 is moved one note to the right. Issue 4776/2 Doc: Usage 1.2: update svg default font info Default svg font-family values are now 'serif' 'sans-serif' and 'monospace' Issue 4776/1 font.scm: better svg defaults for font-family if the user has not set the font, use a meaningful svg default ('serif' 'sans-serif' or 'monospace') for the font-family property Doc: NR - changing-defaults.itely - various improvements Issue 4761 A variety of nitpicks and improvements reported by Federico Bruni Some spelling corrections and typos fixed. The section on \set vs. \override has been edited in an attmpt to simplify the explanation for the reader. Fixed a couple of TODOs. Added a @c that references an open issue to improve the example for pure/unpure containers as the example no longer illustrates what it used to as of LP 2.17.x +

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+157 lines, -122 lines) Patch
M Documentation/notation/changing-defaults.itely View 13 chunks +107 lines, -90 lines 0 comments Download
M Documentation/usage/running.itely View 1 chunk +12 lines, -11 lines 0 comments Download
M python/musicxml.py View 5 chunks +27 lines, -13 lines 0 comments Download
M scm/font.scm View 3 chunks +11 lines, -8 lines 0 comments Download

Messages

Total messages: 1
pkx166h
8 years, 2 months ago (2016-02-28 09:40:07 UTC) #1
Message was sent while issue was closed.
Well I really ballsed that patch upload up didn't I?
Sign in to reply to this message.

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