Minor corrections regarding of the text function.

[inaridarkfox4231]

there are two problems:

1. If you specify CENTER or BOTTOM for the second argument of textAlign without the 4th and 5th arguments of the text function, some lines will disappear.
2. If CENTER is specified for rectMode, some lines may disappear when box is set by the text function.

These fixes are meant to resolve these issues.

Resolves: #5880

Changes:

sampleCode for problem 1

function setup() {
  createCanvas(400, 400);
  background(0);
  noStroke();
  fill(255);

  textSize(18);
  textAlign(CENTER, TOP);
  text("first\nsecond\nthird", 80, 200);
  textAlign(CENTER, BASELINE);
  text("first\nsecond\nthird", 160, 200);
  textAlign(CENTER, BOTTOM);
  text("first\nsecond\nthird", 240, 200);
  textAlign(CENTER, CENTER);
  text("first\nsecond\nthird", 320, 200);

  textSize(14);
  text("TOP", 80, 120);
  text("BASELINE", 160, 120);
  text("BOTTOM", 240, 120);
  text("CENTER", 320, 120);
  stroke(255);
  line(0,200,400,200);
}

left: before change. right: after change.
Screenshots of the change:

sampleCode for problem 2

function setup() {
  createCanvas(400, 400);
  background(0);
  noStroke();
  fill(255);
  rectMode(CENTER);

  textSize(24);
  textAlign(CENTER, BASELINE);
  text("first\nsecond\nthird\nfourth", 50, 200, 100, 400);
  textAlign(CENTER, TOP);
  text("first\nsecond\nthird\nfourth", 150, 200, 100, 400);
  textAlign(CENTER, CENTER);
  text("first\nsecond\nthird\nfourth", 250, 200, 100, 400);
  textAlign(CENTER, BOTTOM);
  text("first\nsecond\nthird\nfourth", 350, 200, 100, 400);

  noFill();
  stroke(255);
  rect(50, 200, 100, 400);
  rect(150, 200, 100, 400);
  rect(250, 200, 100, 400);
  rect(350, 200, 100, 400);
}

left: before change. right: after change.
Screenshots of the change:

PR Checklist

• TypoGraphy

[welcome]

🎉 Thanks for opening this pull request! Please check out our contributing guidelines if you haven't already. And be sure to add yourself to the list of contributors on the readme page!

[dhowe]

Looks very good to me. A bit unusual perhaps to put 2 statements on one line (as in 270), but it seems to pass the linter, so all good

[dhowe]

@raclim looks good to merge - 2 fixes to live bugs with text-alignment

[inaridarkfox4231]

I'm sorry, it would have been better to split it into two lines...is it okay to leave it as is?

[dhowe]

Perhaps update the PR -- just for practice :)

[dhowe]

great

[inaridarkfox4231]

thank you very much for advice! ('ω')

[dhowe]

thank you very much for advice! ('ω')

thanks for the fix!

[Qianqianye]

Thank you @inaridarkfox4231 and @dhowe ✨!