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

Issue 45160043: Cleanup of ugly MI and SOL shaped noteheads (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 4 months ago by Carl P.
Modified:
10 years, 3 months ago
Reviewers:
wl, lemzwerg, carl.d.sorensen
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Cleanup of ugly MI and SOL shaped noteheads 1. Reducing width of MI notehead, for both "regular" (Aiken, etc.) and "small" (Funk/Walker) heads. When Funk/Walker heads were introduced, a special width variable was introduced to both sets and the mi heads were made wider than the other noteheads. This does not seem to be the practice of available examples (Sacred Harp, Southern Harmony books, etc.) and causes issues with alignment, particularly with stems up, and is visually odd. The variables have been removed from the metafont code, making the MI noteheads the same width as the other shaped notes in their respective sets. 2. Changing the SOL notehead to more closely match the normal round notehead (as advertised in the mf code). The previous code uses the shape of the open (half note) notehead, which is visually unappealing. The open notehead has been defined so that it matches visually with both the normal Aiken/Sacred Harp heads, as well as the thin variants (in keeping with observed practice, which uses the regular open roundhead even when thin variants are used).

Patch Set 1 #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -26 lines) Patch
M Documentation/contributor/source-code.itexi View 1 chunk +0 lines, -5 lines 2 comments Download
M mf/feta-noteheads.mf View 8 chunks +19 lines, -21 lines 2 comments Download

Messages

Total messages: 8
Carl P.
Patch for review to make shaped noteheads conform more closely to personally-observed practice.
10 years, 4 months ago (2013-12-23 22:05:12 UTC) #1
lemzwerg
LGTM, codewise. Since I don't use Sacred Harp notation, I can't give further comments w.r.t. ...
10 years, 4 months ago (2013-12-24 00:57:53 UTC) #2
Carl P.
https://codereview.appspot.com/45160043/diff/1/Documentation/contributor/source-code.itexi File Documentation/contributor/source-code.itexi (left): https://codereview.appspot.com/45160043/diff/1/Documentation/contributor/source-code.itexi#oldcode1387 Documentation/contributor/source-code.itexi:1387: First you will see a terminal editor where you ...
10 years, 4 months ago (2013-12-24 01:04:19 UTC) #3
wl_gnu.org
> I think this happened when I updated the source before submitting. > How do ...
10 years, 4 months ago (2013-12-24 01:18:16 UTC) #4
Carl
The extra width on the "ugly" mi notes was a specific design decision that was ...
10 years, 4 months ago (2013-12-24 22:57:46 UTC) #5
Carl P.
On Dec 24, 2013 5:57 PM, <Carl.D.Sorensen@gmail.com> wrote: > > The extra width on the ...
10 years, 4 months ago (2013-12-24 23:22:00 UTC) #6
Carl
On 2013/12/24 23:22:00, Carl P. wrote: > This may well need to happen. As I ...
10 years, 4 months ago (2013-12-31 17:29:53 UTC) #7
Carl P.
10 years, 4 months ago (2013-12-31 17:54:42 UTC) #8
On Tue, Dec 31, 2013 at 12:29 PM,  <Carl.D.Sorensen@gmail.com> wrote:
> On 2013/12/24 23:22:00, Carl P. wrote:
>
>> This may well need to happen. As I said, I've tried a few times over
>
> the
>>
>> last 8 or 9 months to inquire about the mi head particulary, with no
>> response.
>
>
> If you're the only person on the lists who cares, then you should be
> able to have it the way you want it.
>
> I'm curious, though, about the usage in chords.  In particular, it seems
> that chords in Walker notes can't possibly line up, since do is centered
> on the stem.  Are chords commonly used in shape notes?  Every reference
> I've seen has no chords -- only single notes.

Depending upon the style sheet, shaped notes have varying usage in
chords. Even in single-note formats (Sacred Harp, Southern Harmony,
etc.), there are examples of two notes being given for one part
(optional octaves or providing a second note to complete the chord,
typically SOL). For my own work, notes are nearly always written in
chords, unless the notes are less than a third apart or there is a
difference in rhythm. LilyPond's implementation of chords with Walker
is to center the stem if the notehead nearest the stem is DO or to act
normally otherwise.
>
> I'm fine to have this pushed if nobody else objects.  But I do like
> David K.'s suggestion of a message on -user with before and after
> shapes.
>

I have sample PDFs I generated showing before and after, both singly
and in chords. I haven't yet had time to convert/compress those
examples to smaller file sizes for sending to -user, but hope to in
the next couple of days. In the meantime, I'll probably take the patch
off the countdown.

Carl P.
Sign in to reply to this message.

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