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

Implement support for italics #197

Merged
merged 2 commits into from
Sep 6, 2021
Merged

Implement support for italics #197

merged 2 commits into from
Sep 6, 2021

Conversation

WesleyKlop
Copy link
Contributor

Adds a configuration option for displaying information with the italics style.

Fixes #146

Before:
image

After (configured example_variable to use italic instead of underline):
image

@niklasmohrin
Copy link
Collaborator

Thank you for contributing, this looks good! 🚀

Could you add an integration test that checks that italic styling works (or enhance one of the styling tests to style something as italic too)? It should be as easy as picking something in the test config.toml to be rendered as italic and updating the expected results

@WesleyKlop
Copy link
Contributor Author

Sorry for the late reply, will add this when I have some time :)

@WesleyKlop
Copy link
Contributor Author

Well that was easy :P

Copy link
Collaborator

@niklasmohrin niklasmohrin left a comment

Choose a reason for hiding this comment

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

Very nicely done. Thanks a lot!

@niklasmohrin niklasmohrin requested a review from dbrgn August 7, 2021 21:20
@niklasmohrin
Copy link
Collaborator

looks good to me @dbrgn

Copy link
Collaborator

@dbrgn dbrgn left a comment

Choose a reason for hiding this comment

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

Thank you @WesleyKlop!

@dbrgn dbrgn merged commit 8051c31 into tealdeer-rs:master Sep 6, 2021
@WesleyKlop WesleyKlop deleted the implement-italic-styling branch September 6, 2021 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Allow italics styling
4 participants