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

Release #1885

Merged
merged 2 commits into from
Dec 27, 2024
Merged

Release #1885

merged 2 commits into from
Dec 27, 2024

Conversation

wang1212
Copy link
Member

No description provided.

wang1212 and others added 2 commits December 27, 2024 12:00
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link

Walkthrough: This pull request primarily updates the dependencies across multiple packages within the repository, reflecting patch changes. Additionally, it includes a bug fix in the text word wrapping logic to correctly handle newline characters. These changes aim to improve the functionality and maintainability of the codebase.

Changes:

Files Summary
tests/demos/bugfix/textWordWrap.ts Adjusted maxLines from 3 to 1 and wordWrapWidth from 84 to 30 to fix text word wrapping issues.
packages/*/CHANGELOG.md Added entries for new patch versions and updated dependencies.
packages/*/package.json Updated version numbers and dependencies.
packages/g-lite/src/services/TextService.ts Added logic to handle newline characters in text word wrapping.

@@ -388,6 +389,9 @@ export class TextService {
calcWidth(txt) < widthThreshold &&
textCharIndex < chars.length - 1
) {
if (self.isNewline(chars[textCharIndex + 1])) {

Choose a reason for hiding this comment

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

Ensure that the isNewline method correctly identifies newline characters to prevent any unexpected behavior in text wrapping.

@wang1212 wang1212 merged commit e6c803d into next Dec 27, 2024
5 checks passed
@wang1212 wang1212 deleted the release branch December 27, 2024 04:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant