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

Add embed link type #127

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Conversation

shibaobun
Copy link

@shibaobun shibaobun commented Oct 15, 2022

Dependencies

This branch is based on the changes proposed in #126, since it adds some additional test cases testing modes + URL features like brackets.
936be73 shows changes after #126, so that might be easier to review.
However, I can happily repush this branch without #126 improvements if needed!

What

  • Refactors silent boolean into a string for additional link modes
  • Adds additional embed type for embedding images, like ![alt text](https://link.com/image.jpg)

Why

Additional info (optional)

I am very new to typescript, so please let me know if there's anything that can be changed for best practice!
Thank you!

@shibaobun shibaobun force-pushed the link-modes branch 2 times, most recently from ea39d78 to 9d5277b Compare October 15, 2022 18:21
@shibaobun shibaobun changed the title Add additional link modes Add embed link type Oct 15, 2022
@shibaobun shibaobun force-pushed the link-modes branch 2 times, most recently from 45d34a2 to 396abd9 Compare February 5, 2023 16:21
@shibaobun
Copy link
Author

@syuilo this one has also been freshly updated 😄

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.

1 participant