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

PR for :emphasis property in issue#520 #522

Merged
merged 4 commits into from
Jan 4, 2024

Conversation

imlakshay08
Copy link
Contributor

@imlakshay08 imlakshay08 commented Jan 4, 2024

Description

for #520

This PR introduces support for the :emphasis style within the text rendering functionalities of Scarpe. It enables the italicization of text elements such as em, span, and other applicable tags based on Shoes conventions.

Motivation and Context:

  • Feature Enhancement: By incorporating support for the :emphasis style, Scarpe gains the ability to properly italicize text where necessary, improving visual representation and flexibility in text styling.

  • Alignment with Specifications: This change aligns Scarpe's text styling capabilities with the expected behavior specified for the :emphasis style in Shoes. It allows for three settings: "normal," "oblique," and "italic," enabling versatile text formatting.

Checklist

  • Run tests locally

@noahgibbs
Copy link
Collaborator

Looks good. Assuming the tests pass (I expect they will) I'm cool with merging this.

@noahgibbs noahgibbs merged commit 2617853 into scarpe-team:main Jan 4, 2024
1 check passed
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.

2 participants