textToPoints() not handling line-breaks correctly

[dhowe]

textToPoints() inserts a blank character instead of a line-break

For the string 'B\nA', text() (in white) handles linebreak correctly, while textToPoints() does not:

// load font first

function setup() {
  createCanvas(400, 400);
  textFont(font, 100);
  pts = font.textToPoints('B\nA', 0, 0, 100, {
    sampleFactor: 0.4 // increase for more points
  });
  background(0);
  fill(255);
  noLoop()
}

function draw() {
  text('B\nA', 200, 200)
  pts.forEach(p => circle(200 + p.x, 200 + p.y, 3));
}

[sixhat]

Does this happen with other non-printing characters, like '\t', spaces, etc?

[dhowe]

tab and spaces, definitely not
I suppose we will need to catch line break chars (\r, \n etc.) and handle linebreaks manually

[ffd8]

Being able to add \n line breaks to textToPoints() would be huge and oh so useful!
I'm not so sure that opentype.js supports it?

[dhowe]

I don't think so, but that is not required - we would just need to compute the layout for the entire string (as we do with text()), then get the points for each line (after line breaks) separately

Meantime, the following workaround should support most users: pass each line to textToPoints() separately

[Ashley-Y-Lin]

Is anyone currently working on this?

[dhowe]

Not to my knowledge

[Decoy4ever]

Hey, my group and I from the University of Adelaide want to tackle this issue as part of an open-source contribution project. Is the issue still available to for us to tackle?

[davepagurek]

I believe it's still up for grabs! I'll assign it to you.

[munusshih]

Hi guys! I'm working on this issue as part of my GSoC project. I will do a pull request by adopting most of @Decoy4ever 's suggestion here (#6163) and fix this in the future version of p5 👍