Extract hyphen dimensions and/or hyphen glyph from the font

Issue Tracker for LilyPond

Brought to you by: fedelibre, hahnjo, philholmes, tdanielsmusic

#1255 Extract hyphen dimensions and/or hyphen glyph from the font

Status: Started

Owner: Knut Petersen

Labels: None

Needs:

Patch: needs_work

Type: Enhancement

Updated: 2017-10-04

Created: 2010-09-09

Creator: Anonymous

Private: No

Originally created by: *anonymous

Originally created by: RalphBug...@gmail.com

LyricHyphen does not use the glyph of the LyricText font, but instead a custom line markup. Alexander Kobel: "Once I also tried to exchange the LyricHyphen stencil to mimic LyricText, but to no avail - AFAICS, the alignment of the hyphens is more or less hard-coded in the C++ part."

lily/lyric-hyphen.cc: (line 555-556)
"TODO: should extract hyphen dimensions or hyphen glyph from the font."

--

Allow a markup to replace the default LyricHyphen

Solves issue #1255.

Signed-off-by: Knut Petersen knut_petersen@t-online.de

http://codereview.appspot.com/325470043

Discussion

Google Importer - 2010-09-09

Originally posted by: brownian.box@gmail.com

Some discussion here:
http://lists.gnu.org/archive/html/lilypond-user/2010-09/msg00063.html

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2010-09-09

Originally posted by: perpeduu...@gmail.com

Before they get lost on my webspace: here are the digicam photos of some scores I checked out and referenced in <http://lists.gnu.org/archive/html/lilypond-user/2010-09/msg00128.html>.

baerenreiter.jpg

breitkopf.jpg

carus.jpg

peters.jpg

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2010-10-02

Originally posted by: percival.music.ca@gmail.com

(No comment was entered for this change.)

Labels: -Type-Other Type-Enhancement Priority-Low

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Simon Albrecht - 2016-09-27

summary: Add prefix to lyrics - extract hyphen dimensions or hyphen glyph from the font --> Extract hyphen dimensions and/or hyphen glyph from the font

Description has changed:

Diff:

Needs: -->

Patch: -->
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Simon Albrecht - 2016-09-27

The current behaviour also has its pros. I think we should allow for both by providing different stencil procedures, like ly:lyric-hyphen::print-flexible and ly:lyric-hyphen::print-from-font (names to be discussed). In the latter case, text-interface properties would be read.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Knut Petersen - 2017-09-14

The attached patch solves the problem.

0001-Allow-a-markup-to-replace-the-default-LyricHyphen.patch

issue1255test.ly

issue1255test.pdf

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Knut Petersen - 2017-09-14

Allow a markup to replace the default LyricHyphen

Solves issue #1255.

Signed-off-by: Knut Petersen knut_petersen@t-online.de

http://codereview.appspot.com/325470043

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2017-09-14

assigned_to: pkx166h

Needs: -->

Type: -->
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Knut Petersen - 2017-09-14

Version 2 of issue #1255 patch

http://codereview.appspot.com/325470043

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Knut Petersen - 2017-09-14

Test file changed to support the V2 syntax.

issue1255test.ly

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Description has changed:

Diff:

--- old
+++ new
@@ -6,3 +6,13 @@

 lily/lyric-hyphen.cc: \(line 555-556\)
  "TODO: should extract hyphen dimensions or hyphen glyph from the font."
+ 
+ --
+ 
+ Allow a markup to replace the default LyricHyphen
+
+Solves issue #1255.
+
+Signed-off-by: Knut Petersen knut_petersen@t-online.de
+
+http://codereview.appspot.com/325470043

Needs: -->
Type: -->

Anonymous - 2017-09-14

Patch: new --> review
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2017-09-14

Passes make, make check and a full make doc.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2017-09-16

Patch: review --> countdown

Type: --> Enhancement
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2017-09-16

Patch on countdown for September 19th

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Knut Petersen - 2017-09-19

It is more important to work on the ghostscript issue right now.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2017-09-20

I cannot see that getting resolved any time 'soon' so do you want to push this or not?

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2017-09-23

assigned_to: pkx166h --> Knut Petersen
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2017-09-23

Setting back to needs work.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2017-09-23

Patch: countdown --> needs_work
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Knut Petersen - 2017-10-04

Issue 5210 has been opened. The proposed patch does not entirely solve this issue, so it is better to keep this open and use a new issue.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link: