Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

multiline text #327

Closed
wants to merge 4 commits into from
Closed

multiline text #327

wants to merge 4 commits into from

Conversation

Fil
Copy link
Contributor

@Fil Fil commented Apr 15, 2021

(Is it me or why does it have to be so verbose, I'm repeating x and dx for every line, and dy changes between 1st and next lines, and not to forget putting in a space for empty lines…?)

demo https://observablehq.com/d/9be0dacbb5c59c92

closes #313

Fil added 2 commits April 15, 2021 18:59
(Is it me or why does it have to be so verbose, I'm repeating x nd dx for every line, and dy changes between 1st and next lines…?)

closes #313
@Fil Fil requested a review from mbostock April 15, 2021 17:08
@mbostock
Copy link
Member

mbostock commented Apr 15, 2021

This one is a little complicated… Edit: meaning there are some nuances here about how exactly this should behave and what options we need to provide support for. I’m just signaling that I don’t think this is easy to land as-is but I don’t have time just now to enumerate the nuances. But I’ll try to get to it.

@mbostock
Copy link
Member

Closing in favor of #677.

@mbostock mbostock closed this Jan 19, 2022
@Fil Fil deleted the fil/multiline-text-313 branch January 19, 2022 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Plot.text could be smart with newlines
2 participants