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

#47 Fixed the function to wrap img in figure tag #71

Merged
merged 1 commit into from
Feb 22, 2021

Conversation

akabekobeko
Copy link
Member

#47 対応です。

  • Markdown において画像が単独行として記述された場合のみ、<figure> タグへ包みます
    • <img> が兄弟要素を持たない <p> の子であることを条件とします
    • 既存の「キャプションが設定されている」という条件は維持し、新たなものと両方を満たしている必要があります
  • 画像に属性が指定されていた場合...
    • id なら <figure> へ移動
    • その他は <img> 固有の属性 以外を <figure> へコピーします
  • <figure> 処理は <pre> も対象としているのですが、こちらは既存実装のままです

@akabekobeko akabekobeko added bug Something isn't working enhancement labels Feb 22, 2021
@akabekobeko akabekobeko self-assigned this Feb 22, 2021
@akabekobeko
Copy link
Member Author

@MurakamiShinyu レビューをお願いします。

Copy link
Member

@MurakamiShinyu MurakamiShinyu left a comment

Choose a reason for hiding this comment

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

💯

@akabekobeko akabekobeko merged commit ca63553 into master Feb 22, 2021
@akabekobeko akabekobeko deleted the fix-invalid-figure-tag branch February 22, 2021 11:29
@akabekobeko
Copy link
Member Author

@MurakamiShinyu レビューありがとうございました。merge しました。

@akabekobeko akabekobeko added this to the v1.0.0 milestone Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants