On Tue, May 26, 2009 at 07:51:22AM +0100, Trevor Daniels wrote:
>
> Patrick McCarty wrote Monday, May 25, 2009 11:43 PM
>
>> On Mon, May 25, 2009 at 03:26:42PM -0700, Patrick McCarty wrote:
>>> On Mon, May 25, 2009 at 2:49 PM, Trevor Daniels  
>>> <t.dani...@treda.co.uk> wrote:
>>> >
>>> > The lines in the html version of this section of the
>>> > Notation Reference are significantly longer than any
>>> > other, so long I have to scroll horizontally to read them.
>>> >
>>> > Does anyone else see this, or is it some subtle problem
>>> > with IE?
>>>
>>> IIRC, if the line lengths for any <pre> section are too wide, IE
>>> recalculates the width of the div to accomodate the longest line on
>>> the page.  Other browsers don't behave this way (as far as I
>>> remember).
>>>
>>> I suspect the snippet "Non-default tuplet numbers" in the Tuplets
>>> section is the culprit.  If the values of #'text are moved to  
>>> separate
>>> lines, that should solve the problem.
>>
>> And here is a patch.
>
> Patrick
>
> Many thanks, I'm sure your analysis is correct.  But like Carl I find
> the patch doesn't apply, although I didn't see his particular problems.
> (Your patch also has 4 lines with trailing white-space, but that's not
> the problem.)  Do you want to try again, or should I or Carl simply fix
> it ourselves now you've identified the cause?

Hmm, I can't figure this out.

I just applied my original patch to master (cleanly), and checked the
snippet (both in input/new and input/lsr) for trailing whitespace.
There was only one line with trailing whitespace.  Before applying the
patch, the two files still only containing one line with trailing
whitespace.

However, the *patch* contained trailing whitespace on some of the
empty lines as well, but it still applied cleanly for me.

Maybe this is a bug in "git format-patch" in my installed git verson
(1.6.3.1).  It seems odd that "git format-patch" would add trailing
whitespace.

I created a fresh patch and manually removed all of the trailing
whitespace from it, except for the one in the texidoc which I fixed.
Let me know if this one works.

Thanks,
Patrick
>From 785a948a1c612b58061903068479fbde245f6cec Mon Sep 17 00:00:00 2001
From: Patrick McCarty <pnor...@gmail.com>
Date: Mon, 25 May 2009 15:38:23 -0700
Subject: [PATCH] Docs: Shorten line lengths for a snippet

Signed-off-by: Patrick McCarty <pnor...@gmail.com>
---
 input/lsr/non-default-tuplet-numbers.ly |   25 +++++++++++++++++--------
 input/new/non-default-tuplet-numbers.ly |   25 +++++++++++++++++--------
 2 files changed, 34 insertions(+), 16 deletions(-)

diff --git a/input/lsr/non-default-tuplet-numbers.ly 
b/input/lsr/non-default-tuplet-numbers.ly
index c910ff6..9474379 100644
--- a/input/lsr/non-default-tuplet-numbers.ly
+++ b/input/lsr/non-default-tuplet-numbers.ly
@@ -37,7 +37,7 @@ au chiffre.

   texidoc = "
 LilyPond also provides formatting functions to print tuplet numbers different
-than the actual fraction, as well as to append a note value to the tuplet 
+than the actual fraction, as well as to append a note value to the tuplet
 number or tuplet fraction.
 "
   doctitle = "Non-default tuplet numbers"
@@ -45,21 +45,30 @@ number or tuplet fraction.


 \relative c'' {
-  \once \override TupletNumber #'text = 
#(tuplet-number::non-default-tuplet-denominator-text 7)
+  \once \override TupletNumber #'text =
+    #(tuplet-number::non-default-tuplet-denominator-text 7)
   \times 2/3  { c4. c4. c4. c4. }
-  \once \override TupletNumber #'text = 
#(tuplet-number::non-default-tuplet-fraction-text 12 7)
+  \once \override TupletNumber #'text =
+    #(tuplet-number::non-default-tuplet-fraction-text 12 7)
   \times 2/3  { c4. c4. c4. c4. }
   \once \override TupletNumber #'text =
-    #(tuplet-number::append-note-wrapper 
(tuplet-number::non-default-tuplet-fraction-text 12 7) "8")
+    #(tuplet-number::append-note-wrapper
+       (tuplet-number::non-default-tuplet-fraction-text 12 7) "8")
   \times 2/3  { c4. c4. c4. c4. }

-  \once \override TupletNumber #'text = #(tuplet-number::append-note-wrapper 
tuplet-number::calc-denominator-text "4")
+  \once \override TupletNumber #'text =
+    #(tuplet-number::append-note-wrapper
+       tuplet-number::calc-denominator-text "4")
   \times 2/3  { c8 c8 c8 c8 c8 c8 }
-  \once \override TupletNumber #'text = #(tuplet-number::append-note-wrapper 
tuplet-number::calc-fraction-text "4")
+  \once \override TupletNumber #'text =
+    #(tuplet-number::append-note-wrapper
+       tuplet-number::calc-fraction-text "4")
   \times 2/3  { c8 c8 c8 c8 c8 c8 }

-  \once \override TupletNumber #'text = #(tuplet-number::fraction-with-notes 
"4." "8")
+  \once \override TupletNumber #'text =
+    #(tuplet-number::fraction-with-notes "4." "8")
   \times 2/3  { c4. c4. c4. c4. }
-  \once \override TupletNumber #'text = 
#(tuplet-number::non-default-fraction-with-notes 12 "8" 4 "4")
+  \once \override TupletNumber #'text =
+    #(tuplet-number::non-default-fraction-with-notes 12 "8" 4 "4")
   \times 2/3  { c4. c4. c4. c4. }
 }
diff --git a/input/new/non-default-tuplet-numbers.ly 
b/input/new/non-default-tuplet-numbers.ly
index 29b109c..9cd2f5b 100644
--- a/input/new/non-default-tuplet-numbers.ly
+++ b/input/new/non-default-tuplet-numbers.ly
@@ -5,28 +5,37 @@

   texidoc = "
 LilyPond also provides formatting functions to print tuplet numbers different
-than the actual fraction, as well as to append a note value to the tuplet 
+than the actual fraction, as well as to append a note value to the tuplet
 number or tuplet fraction.
 "
   doctitle = "Non-default tuplet numbers"
 }

 \relative c'' {
-  \once \override TupletNumber #'text = 
#(tuplet-number::non-default-tuplet-denominator-text 7)
+  \once \override TupletNumber #'text =
+    #(tuplet-number::non-default-tuplet-denominator-text 7)
   \times 2/3  { c4. c4. c4. c4. }
-  \once \override TupletNumber #'text = 
#(tuplet-number::non-default-tuplet-fraction-text 12 7)
+  \once \override TupletNumber #'text =
+    #(tuplet-number::non-default-tuplet-fraction-text 12 7)
   \times 2/3  { c4. c4. c4. c4. }
   \once \override TupletNumber #'text =
-    #(tuplet-number::append-note-wrapper 
(tuplet-number::non-default-tuplet-fraction-text 12 7) "8")
+    #(tuplet-number::append-note-wrapper
+       (tuplet-number::non-default-tuplet-fraction-text 12 7) "8")
   \times 2/3  { c4. c4. c4. c4. }

-  \once \override TupletNumber #'text = #(tuplet-number::append-note-wrapper 
tuplet-number::calc-denominator-text "4")
+  \once \override TupletNumber #'text =
+    #(tuplet-number::append-note-wrapper
+       tuplet-number::calc-denominator-text "4")
   \times 2/3  { c8 c8 c8 c8 c8 c8 }
-  \once \override TupletNumber #'text = #(tuplet-number::append-note-wrapper 
tuplet-number::calc-fraction-text "4")
+  \once \override TupletNumber #'text =
+    #(tuplet-number::append-note-wrapper
+       tuplet-number::calc-fraction-text "4")
   \times 2/3  { c8 c8 c8 c8 c8 c8 }

-  \once \override TupletNumber #'text = #(tuplet-number::fraction-with-notes 
"4." "8")
+  \once \override TupletNumber #'text =
+    #(tuplet-number::fraction-with-notes "4." "8")
   \times 2/3  { c4. c4. c4. c4. }
-  \once \override TupletNumber #'text = 
#(tuplet-number::non-default-fraction-with-notes 12 "8" 4 "4")
+  \once \override TupletNumber #'text =
+    #(tuplet-number::non-default-fraction-with-notes 12 "8" 4 "4")
   \times 2/3  { c4. c4. c4. c4. }
 }
--
1.6.3.1

_______________________________________________
lilypond-user mailing list
lilypond-user@gnu.org
http://lists.gnu.org/mailman/listinfo/lilypond-user

Reply via email to