-
Notifications
You must be signed in to change notification settings - Fork 309
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
Add letter-spacing
property to RCSS
#429
Add letter-spacing
property to RCSS
#429
Conversation
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.
You've figured your way around the code base very well. Most of my comments are relatively minor fixups.
Could you add a VisualTest as well? Something that shows the output with different letter-spacing. Maybe something with line-breaks or other edge-cases too?
To get started, you could base the test on another test, such as Tests/Data/VisualTests/word_break.rml
.
Just an FYI for the future, you can change the base branch of PRs on GitHub itself. There shouldn't be a need to open new PRs, no worries though.
Good thoughts, thank you! |
23b3911
to
a90b154
Compare
Hey, I decided to finish this one up myself, including a visual test, and some efficiency considerations. Thanks for the pull request! |
Related to #426