-
-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
[FR] Support markdown to italicize text #1059
[FR] Support markdown to italicize text #1059
Comments
I would like to work on this issue! |
Thanks! Assigned to you. The mentor on this issue is @LucasXu0 . If you have any questions, please feel free to reach him out on Discord. |
Hey @annieappflowy, can I try working on this? |
Hi, @JRS296. Sure. We have implemented |
Cool, getting to work right away |
@JRS296 Hey JRS, if you need help feel free to let me know. I have this on a commit if you want some reference. |
This commit (AppFlowy-IO/appflowy-editor@b6a2803) for the italic shortcut. I will try during next week to see if I can replicate your issue on my Windows machine, currently away from home with my MacOS laptop. Cheers |
Hey @Xazin , managed to fix the dev environment, it was due to the AntiVirus interfering with the build. Has this Feature request been implemented with your recent commit? If so, I'll start work on something else 👍 |
I haven't made the PR, but I can do that later today, if you want to complete it feel free to do so. (You can take my commit and just add a test, and it's done) |
Resolved by #2115 |
_ works but * doesn't work |
I'll have a look into it 👍 |
This must be due to the fact that Or so I would assume at least.. |
|
To italicize text, add one asterisk or underscore before and after a word or phrase. To italicize the middle of a word for emphasis, add one asterisk without spaces around the letters.
Reference:
The text was updated successfully, but these errors were encountered: