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

Fixing kerning para style #542

Merged
merged 3 commits into from
Feb 8, 2024
Merged

Conversation

vikas-chaudhary-2802
Copy link
Contributor

Description

I have added the kerning para style --> #520
This PR introduces a new feature to the Scarpe library - a kerning para style. Kerning is an important aspect of typography that adjusts the spacing between characters to achieve a visually pleasing result. This feature will allow users to apply kerning to their paragraphs, enhancing the visual appeal of their text.

Image(if needed, helps for a faster review)

Screenshot 2024-02-08 at 10 18 03 PM

Checklist

  • [Yes] Run tests locally

@noahgibbs
Copy link
Collaborator

Looks good!

@noahgibbs
Copy link
Collaborator

Looks like you'll need to generate HTML fixtures, since you added a new example. You should be able to do that by running "rake test:regenerate_html_fixtures" and adding the generated file to this PR.

vikas-chaudhary-2802 and others added 2 commits February 9, 2024 00:45
@noahgibbs noahgibbs merged commit fd97618 into scarpe-team:main Feb 8, 2024
1 check passed
@@ -3,7 +3,7 @@
class Shoes
class Para < Shoes::Drawable
include FontHelper
shoes_styles :text_items, :size, :family, :font_weight, :font, :font_variant, :emphasis
shoes_styles :text_items, :size, :family, :font_weight, :font, :font_variant, :emphasis, :kerning
Copy link
Collaborator

Choose a reason for hiding this comment

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

@noahgibbs when we get these new excellent features; should we add them to manual.md you think?

Unless I'm being silly and kerning was an existing thing in manual.md 😅

Copy link
Collaborator

Choose a reason for hiding this comment

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

Kerning is in there already! We just only had it work recently :-)

There's a surprising amount of Shoes functionality we're still adding.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants