-
-
Notifications
You must be signed in to change notification settings - Fork 377
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
fix(block): Fixed title_style not rendered #363
Conversation
3ccc39f
to
1922b9e
Compare
Codecov Report
@@ Coverage Diff @@
## main #363 +/- ##
==========================================
+ Coverage 84.76% 84.99% +0.23%
==========================================
Files 40 40
Lines 8603 8686 +83
==========================================
+ Hits 7292 7383 +91
+ Misses 1311 1303 -8
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR and welcome to Ratatui.
Generally LGTM. I had a nitpick and a preference that has to do with keeping tests simple. Can be merged as-is or updated to address the feedback (both are not huge points).
I merged #351 now, and took a look at this visually and it acts as expected.: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - Thanks for the fix. We appreciate your effort.
Merging this even though checks are failing as the failures are due to the MSRV issue. |
This PR fixes #349 the easy way (see issue). I think the complex fix requires another separate PR.
I added a unit test and made sure it didn't pass prior to this modification. I added a lot of test cases because there are three separate places where this bug can occur.
I didn't change the corresponding example to demonstrate the behavior as there is already an opened PR that will likely be merged.
I didn't want to create a new spans (or lines) because in the future they might contain more than just
content
andstyle
and this would lead to some half cloned spans. Thus I preferred cloning and modifying them locally.