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

Sentence-based line breaks for annotations #3514

Merged

Conversation

henrikt-ma
Copy link
Collaborator

Fixes #3512. The fix is the same as in #3513, but with separate transition to sentence-based line breaks.

@henrikt-ma henrikt-ma force-pushed the bugfix/connection-text-filledshape branch from d11f53b to b5e84bf Compare June 3, 2024 09:06
@henrikt-ma
Copy link
Collaborator Author

After rebasing on top of the changes from #3513, only transition to sentence-based line breaks remain. Even though the issue has already been fixed, I think we should take this opportunity to get the line breaks right in the chapter.

@HansOlsson
Copy link
Collaborator

After rebasing on top of the changes from #3513, only transition to sentence-based line breaks remain. Even though the issue has already been fixed, I think we should take this opportunity to get the line breaks right in the chapter.

Ok, but then the name of the PR should then be changed.

Copy link
Collaborator

@HansOlsson HansOlsson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rename PR as it is just line-break changes.

@HansOlsson HansOlsson changed the title Remove forgotten mention of deprecated inheritance from FilledShape Sentence-based line breaks for annotations Jun 11, 2024
@HansOlsson
Copy link
Collaborator

I changed the name of the PR, please confirm @henrikt-ma

@HansOlsson HansOlsson self-requested a review June 11, 2024 11:57
Copy link
Collaborator

@HansOlsson HansOlsson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems ok now, with new name

@henrikt-ma
Copy link
Collaborator Author

I changed the name of the PR, please confirm @henrikt-ma

It's a great new name.

@HansOlsson HansOlsson merged commit 1ce1501 into modelica:master Jun 11, 2024
1 check passed
@henrikt-ma henrikt-ma deleted the bugfix/connection-text-filledshape branch June 11, 2024 13:59
@HansOlsson HansOlsson added the M37 For pull requests merged into Modelica 3.7 label Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
M37 For pull requests merged into Modelica 3.7
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing mention of deprecated differences between the two Text-annotations
2 participants