Skip to content
New issue

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

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

Already on GitHub? # to your account

Support clearing spans #315

Merged
merged 1 commit into from
Aug 11, 2024

Conversation

Goooler
Copy link
Contributor

@Goooler Goooler commented Jul 29, 2024

Closes #75.

@Goooler Goooler marked this pull request as draft July 29, 2024 04:29
@Goooler Goooler marked this pull request as ready for review July 29, 2024 05:39
Comment on lines +228 to +231
// Clear all spans.
richTextState.clearSpanStyles(TextRange(0, 17))

assertEquals(defaultSpan, richTextState.currentSpanStyle)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This failed, looks like a new issue:

Expected :SpanStyle(color=Color(0.0, 0.0, 0.0, 0.0, None), brush=null, alpha=NaN, fontSize=Unspecified, fontWeight=null, fontStyle=null, ...

Actual   :SpanStyle(color=Color(0.0, 0.0, 0.0, 0.0, None), brush=null, alpha=NaN, fontSize=Unspecified, fontWeight=null, fontStyle=Italic, ...

@MohamedRejeb
Copy link
Owner

Thanks for your contribution!
I will check this soon, I've been a bit busy lately.

@MohamedRejeb MohamedRejeb changed the base branch from main to clear-all-spans August 11, 2024 09:52
@MohamedRejeb MohamedRejeb merged commit e3c273c into MohamedRejeb:clear-all-spans Aug 11, 2024
3 of 4 checks passed
@Goooler Goooler deleted the clear-all-spans branch August 11, 2024 09:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

How to clear all formatting
2 participants