abcjs 6.4.2 → 6.4.4

package/dist/abcjs-basic-min.js.LICENSE

@@ -1,5 +1,5 @@
 /**!
-Copyright (c) 2009-2023 Paul Rosen and Gregory Dyke
+Copyright (c) 2009-2024 Paul Rosen and Gregory Dyke
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

package/dist/abcjs-basic.js

@@ -18,7 +18,7 @@ return /******/ (function() { // webpackBootstrap
 /***/ (function(module, __unused_webpack_exports, __webpack_require__) {
 
 /**!
-Copyright (c) 2009-2023 Paul Rosen and Gregory Dyke
+Copyright (c) 2009-2024 Paul Rosen and Gregory Dyke
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -1667,6 +1667,7 @@ var Tune = function Tune() {
   }
   this.getBpm = function (tempo) {
     var bpm;
+    if (!tempo) tempo = this.metaText ? this.metaText.tempo : null;
     if (tempo) {
       bpm = tempo.bpm;
       var beatLength = this.getBeatLength();
@@ -6547,8 +6548,7 @@ MusicParser.prototype.parseMusic = function (line) {
           if (el.chord !== undefined) bar.chord = el.chord;
           if (bar.startEnding && multilineVars.barFirstEndingNum === undefined) multilineVars.barFirstEndingNum = multilineVars.currBarNumber;else if (bar.startEnding && bar.endEnding && multilineVars.barFirstEndingNum) multilineVars.currBarNumber = multilineVars.barFirstEndingNum;else if (bar.endEnding) multilineVars.barFirstEndingNum = undefined;
           if (bar.type !== 'bar_invisible' && multilineVars.measureNotEmpty) {
-            var isFirstVoice = multilineVars.currentVoice === undefined || multilineVars.currentVoice.staffNum === 0 && multilineVars.currentVoice.index === 0;
-            if (isFirstVoice) {
+            if (isFirstVoice()) {
               multilineVars.currBarNumber++;
               if (multilineVars.barNumbers && multilineVars.currBarNumber % multilineVars.barNumbers === 0) bar.barNumber = multilineVars.currBarNumber;
             }
@@ -6743,6 +6743,7 @@ MusicParser.prototype.parseMusic = function (line) {
               if (el.startTie !== undefined) el.pitches[0].startTie = el.startTie;
             } else {
               el.rest = core.rest;
+              if (core.rest.type === 'multimeasure' && isFirstVoice()) multilineVars.currBarNumber += core.rest.text - 1; // The minus one is because the measure with the rest is already counted once normally.
               if (core.endSlur !== undefined) el.endSlur = core.endSlur;
               if (core.endTie !== undefined) el.rest.endTie = core.endTie;
               if (core.startSlur !== undefined) el.startSlur = core.startSlur;
@@ -6836,15 +6837,30 @@ var letter_to_chord = function letter_to_chord(line, i) {
       chord[1] = chord[1].substring(1);
       chord[2] = 'right';
     } else if (chord[0] > 0 && chord[1].length > 0 && chord[1][0] === '@') {
-      // @-15,5.7
+      // @-15,5.7		
       chord[1] = chord[1].substring(1);
       var x = tokenizer.getFloat(chord[1]);
-      if (x.digits === 0) warn("Missing first position in absolutely positioned annotation.", line, i);
+      if (x.digits === 0) {
+        warn("Missing first position in absolutely positioned annotation.", line, i);
+        chord[1] = chord[1].replace("@", "");
+        chord[2] = 'above';
+        return chord;
+      }
       chord[1] = chord[1].substring(x.digits);
-      if (chord[1][0] !== ',') warn("Missing comma absolutely positioned annotation.", line, i);
+      if (chord[1][0] !== ',') {
+        warn("Missing comma absolutely positioned annotation.", line, i);
+        chord[1] = chord[1].replace("@", "");
+        chord[2] = 'above';
+        return chord;
+      }
       chord[1] = chord[1].substring(1);
       var y = tokenizer.getFloat(chord[1]);
-      if (y.digits === 0) warn("Missing second position in absolutely positioned annotation.", line, i);
+      if (y.digits === 0) {
+        warn("Missing second position in absolutely positioned annotation.", line, i);
+        chord[1] = chord[1].replace("@", "");
+        chord[2] = 'above';
+        return chord;
+      }
       chord[1] = chord[1].substring(y.digits);
       var ws = tokenizer.skipWhiteSpace(chord[1]);
       chord[1] = chord[1].substring(ws);
@@ -7188,8 +7204,7 @@ MusicParser.prototype.startNewLine = function () {
       if (params.currentVoice.staffNum === multilineVars.voices[voices[mv]].staffNum && params.currentVoice.index === multilineVars.voices[voices[mv]].index) params.currentVoiceName = voices[mv];
     }
   }
-  var isFirstVoice = multilineVars.currentVoice === undefined || multilineVars.currentVoice.staffNum === 0 && multilineVars.currentVoice.index === 0;
-  if (multilineVars.barNumbers === 0 && isFirstVoice && multilineVars.currBarNumber !== 1) params.barNumber = multilineVars.currBarNumber;
+  if (multilineVars.barNumbers === 0 && isFirstVoice() && multilineVars.currBarNumber !== 1) params.barNumber = multilineVars.currBarNumber;
   tuneBuilder.startNewLine(params);
   if (multilineVars.key.impliedNaturals) delete multilineVars.key.impliedNaturals;
   multilineVars.partForNextLine = {};
@@ -7485,6 +7500,9 @@ var getBrokenRhythm = function getBrokenRhythm(line, index) {
   }
   return null;
 };
+function isFirstVoice() {
+  return multilineVars.currentVoice === undefined || multilineVars.currentVoice.staffNum === 0 && multilineVars.currentVoice.index === 0;
+}
 module.exports = MusicParser;
 
 /***/ }),
@@ -9558,7 +9576,7 @@ var TuneBuilder = function TuneBuilder(tune) {
   };
   this.getCurrentVoice = function () {
     //console.log("getCurrentVoice", tune.lineNum)
-    var currLine = tune.lines[tune.lineNum];
+    var currLine = getPrevMusicLine(tune.lines, tune.lineNum);
     if (!currLine) return null;
     var currStaff = currLine.staff[tune.staffNum];
     if (!currStaff) return null;
@@ -9981,6 +9999,14 @@ function wrapMusicLines(lines, barsperstaff) {
   }
   return false;
 }
+function getPrevMusicLine(lines, currentLine) {
+  // If the current line doesn't have music, search backwards until one is found.
+  while (currentLine >= 0) {
+    if (lines[currentLine].staff) return lines[currentLine];
+    currentLine--;
+  }
+  return null;
+}
 function getNextMusicLine(lines, currentLine) {
   currentLine++;
   while (lines.length > currentLine) {
@@ -11512,7 +11538,7 @@ var pitchesToPerc = __webpack_require__(/*! ./pitches-to-perc */ "./src/synth/pi
     var shortestNote = durationRounded(1.0 / 32);
     switch (noteModification) {
       case "trill":
-        var note = 1;
+        var note = 2;
         while (runningDuration > 0) {
           currentTrack.push({
             cmd: 'note',
@@ -11524,7 +11550,7 @@ var pitchesToPerc = __webpack_require__(/*! ./pitches-to-perc */ "./src/synth/pi
             instrument: currentInstrument,
             style: 'decoration'
           });
-          note = note === 1 ? 0 : 1;
+          note = note === 2 ? 0 : 2;
           runningDuration -= shortestNote;
           start += shortestNote;
         }
@@ -11544,7 +11570,7 @@ var pitchesToPerc = __webpack_require__(/*! ./pitches-to-perc */ "./src/synth/pi
         start += shortestNote;
         currentTrack.push({
           cmd: 'note',
-          pitch: p.pitch + 1,
+          pitch: p.pitch + 2,
           volume: p.volume,
           start: start,
           duration: shortestNote,
@@ -11579,7 +11605,7 @@ var pitchesToPerc = __webpack_require__(/*! ./pitches-to-perc */ "./src/synth/pi
         start += shortestNote;
         currentTrack.push({
           cmd: 'note',
-          pitch: p.pitch - 1,
+          pitch: p.pitch - 2,
           volume: p.volume,
           start: start,
           duration: shortestNote,
@@ -11600,10 +11626,10 @@ var pitchesToPerc = __webpack_require__(/*! ./pitches-to-perc */ "./src/synth/pi
         });
         break;
       case "turn":
-        shortestNote = p.duration / 5;
+        shortestNote = p.duration / 4;
         currentTrack.push({
           cmd: 'note',
-          pitch: p.pitch,
+          pitch: p.pitch + 2,
           volume: p.volume,
           start: start,
           duration: shortestNote,
@@ -11613,7 +11639,7 @@ var pitchesToPerc = __webpack_require__(/*! ./pitches-to-perc */ "./src/synth/pi
         });
         currentTrack.push({
           cmd: 'note',
-          pitch: p.pitch + 1,
+          pitch: p.pitch,
           volume: p.volume,
           start: start + shortestNote,
           duration: shortestNote,
@@ -11623,7 +11649,7 @@ var pitchesToPerc = __webpack_require__(/*! ./pitches-to-perc */ "./src/synth/pi
         });
         currentTrack.push({
           cmd: 'note',
-          pitch: p.pitch,
+          pitch: p.pitch - 1,
           volume: p.volume,
           start: start + shortestNote * 2,
           duration: shortestNote,
@@ -11633,7 +11659,7 @@ var pitchesToPerc = __webpack_require__(/*! ./pitches-to-perc */ "./src/synth/pi
         });
         currentTrack.push({
           cmd: 'note',
-          pitch: p.pitch + 1,
+          pitch: p.pitch,
           volume: p.volume,
           start: start + shortestNote * 3,
           duration: shortestNote,
@@ -11641,15 +11667,6 @@ var pitchesToPerc = __webpack_require__(/*! ./pitches-to-perc */ "./src/synth/pi
           instrument: currentInstrument,
           style: 'decoration'
         });
-        currentTrack.push({
-          cmd: 'note',
-          pitch: p.pitch,
-          volume: p.volume,
-          start: start + shortestNote * 4,
-          duration: shortestNote,
-          gap: 0,
-          instrument: currentInstrument
-        });
         break;
       case "roll":
         while (runningDuration > 0) {
@@ -13395,6 +13412,14 @@ ChordTrack.prototype.interpretChord = function (name) {
   }
   bass += chordTranspose;
 
+  // MAE 31 Aug 2024 - For visual transpose backup range issue
+  // If transposed below A or above G, bring it back in the normal backup range
+  if (bass < 33) {
+    bass += 12;
+  } else if (bass > 44) {
+    bass -= 12;
+  }
+
   // MAE 17 Jun 2024 - Supporting octave shifted bass and chords
   var unshiftedBass = bass;
   bass += this.bassOctaveShift * 12;
@@ -18220,7 +18245,8 @@ AbstractEngraver.prototype.addMeasureNumber = function (number, abselem) {
   if (abselem.isClef)
     // If this is a clef rather than bar line, then the number shouldn't be centered because it could overlap the left side. This is an easy way to let it be centered but move it over, too.
     dx += measureNumDim.width / 2;
-  var vert = measureNumDim.width > 10 && abselem.abcelem.type === "treble" ? 13 : 11;
+  // MAE 1 Oct 2024 - Change 13 to 13.5 since previously bar numbers were very slightly overlapping the top of the clef
+  var vert = measureNumDim.width > 10 && abselem.abcelem.type === "treble" ? 13.5 : 11;
   abselem.addFixed(new RelativeElement(number, dx, measureNumDim.width, vert + measureNumDim.height / spacing.STEP, {
     type: "barNumber",
     dim: this.getTextSize.attr("measurefont", 'bar-number')
@@ -19485,10 +19511,18 @@ AbsoluteElement.prototype.setLimit = function (member, child) {
   if (!this.specialY[member]) this.specialY[member] = child[member];else this.specialY[member] = Math.max(this.specialY[member], child[member]);
 };
 AbsoluteElement.prototype._addChild = function (child) {
-  //	console.log("Relative:",child);
+  //  console.log("Relative:",child);
+
+  // MAE 30 Sep 2024 - To avoid extra space for chords if there is only a bar number on the clef
+  var okToPushTop = true;
+  if (this.abcelem.el_type == "clef" && child.type == "barNumber") {
+    okToPushTop = false;
+  }
   child.parent = this;
   this.children[this.children.length] = child;
-  this.pushTop(child.top);
+  if (okToPushTop) {
+    this.pushTop(child.top);
+  }
   this.pushBottom(child.bottom);
   this.setLimit('tempoHeightAbove', child);
   this.setLimit('partHeightAbove', child);
@@ -20602,6 +20636,30 @@ TieElem.prototype.avoidCollisionAbove = function () {
     if (maxInnerHeight > this.startY && maxInnerHeight > this.endY) this.startY = this.endY = maxInnerHeight - 1;
   }
 };
+TieElem.prototype.getYBounds = function () {
+  var lineStartX = 10; // TODO-PER: I'm not sure where to get this number from but it probably doesn't matter much
+  var lineEndX = 1000; // TODO-PER: I'm not sure where to get this number from but it probably doesn't matter much
+  if (this.isTie) {
+    this.calcTieDirection();
+    this.calcX(lineStartX, lineEndX);
+    this.calcTieY();
+  } else {
+    this.calcSlurDirection();
+    this.calcX(lineStartX, lineEndX);
+    this.calcSlurY();
+  }
+  var top;
+  var bottom;
+  // TODO-PER: It's hard to tell how far the arc is, so I'm just using 3 as the max
+  if (this.above) {
+    bottom = Math.min(this.startY, this.endY);
+    top = bottom + 3;
+  } else {
+    top = Math.min(this.startY, this.endY);
+    bottom = top - 3;
+  }
+  return [top, bottom];
+};
 module.exports = TieElem;
 
 /***/ }),
@@ -25381,6 +25439,14 @@ function setUpperAndLowerVoiceElements(positionY, voice, spacing) {
       case 'EndingElem':
         setUpperAndLowerEndingElements(positionY, abselem);
         break;
+      case 'TieElem':
+        // If a tie element is the highest or lowest thing then space might need to make room for it.
+        var yBounds = abselem.getYBounds();
+        voice.staff.top = Math.max(voice.staff.top, yBounds[0]);
+        voice.staff.top = Math.max(voice.staff.top, yBounds[1]);
+        voice.staff.bottom = Math.min(voice.staff.bottom, yBounds[0]);
+        voice.staff.bottom = Math.min(voice.staff.bottom, yBounds[1]);
+        break;
     }
   }
 }
@@ -26489,7 +26555,7 @@ module.exports = Svg;
   \********************/
 /***/ (function(module) {
 
-var version = '6.4.2';
+var version = '6.4.4';
 module.exports = version;
 
 /***/ })