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

感想を追記 #336

Open
wants to merge 6 commits into
base: gh-pages
Choose a base branch
from
Open

感想を追記 #336

wants to merge 6 commits into from

Conversation

UKONNsoft
Copy link

いまさらですが・・・

Comment on lines +70 to +75
<p class="impression">マージされる前に再コミットしておきます</p>

<p class="impression">具体的なハウツーを簡潔に書かれた記事だと思いました</p>

<p class="impression"><h1>自分で自分にプルリクエスト!?</h1></p>

Copy link
Collaborator

Choose a reason for hiding this comment

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

@UKONNsoft Pull Requestありがとうございます。

感想の中に <h1> の見出しが入ってしまうと表示が崩れてしまいます。この要素は取って欲しいです。

また、 <p class="impression"> でマークアップされた件数をカウントして、寄せられた件数の総数として表示していることから、できれば1つにまとめて頂きたいです。
http://hirocaster.github.io/wdpress69/

Copy link
Collaborator

@raimon49 raimon49 left a comment

Choose a reason for hiding this comment

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

コメント箇所が修正されたらマージしたいと思います。

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