review comments

0xfe · Jan 8, 2022 · aebecd4 · aebecd4
1 parent a8191ab
commit aebecd4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/stave.ts b/src/stave.ts
@@ -310,7 +310,7 @@ export class Stave extends Element {
   }
 
   // Section functions
-  setSection(section: string, y: number, xOffset = 0, fontSize = 12) {
+  setSection(section: string, y: number, xOffset = 0, fontSize = 10) {
     const staveSection = new StaveSection(section, this.x + xOffset, y);
     staveSection.setFontSize(fontSize);
     this.modifiers.push(staveSection);

diff --git a/src/stavesection.ts b/src/stavesection.ts
@@ -12,7 +12,7 @@ export class StaveSection extends StaveModifier {
 
   static TEXT_FONT: Required<FontInfo> = {
     family: Font.SANS_SERIF,
-    size: 12,
+    size: 10,
     weight: FontWeight.BOLD,
     style: FontStyle.NORMAL,
   };
@@ -29,7 +29,7 @@ export class StaveSection extends StaveModifier {
     this.x = x;
     this.shift_x = 0;
     this.shift_y = shift_y;
-    this.setFont('bold 12pt sans-serif');
+    this.resetFont();
   }
 
   setStaveSection(section: string): this {
@@ -67,7 +67,7 @@ export class StaveSection extends StaveModifier {
     let x = this.x + shift_x;
     ctx.beginPath();
     ctx.setLineWidth(2);
-    ctx.rect(x, y + textHeight / 8, width, height);
+    ctx.rect(x, y + textHeight / 4, width, height);
     ctx.stroke();
     x += (width - textWidth) / 2;
     ctx.fillText('' + this.section, x, y + 16);