Skip to content

Add new post#9

Open
yamanetoshi wants to merge 2 commits into
masterfrom
6-yamanetoshi
Open

Add new post#9
yamanetoshi wants to merge 2 commits into
masterfrom
6-yamanetoshi

Conversation

@yamanetoshi

Copy link
Copy Markdown
Contributor

Fix #6

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Front-matterが抜けてて、レイアウトがあたってないようです。
レイアウトあたらないと<meta charset="utf-8">タグが入らないので、手元で確認したところ文字化けしてしまいました 🙈

@yamanetoshi

Copy link
Copy Markdown
Contributor Author

Front-matter 追加してみました。よく考えたら動作確認せずに push してますねorz

一応ローカルで確認したところでは文字化けは出てないようです。記事追加のルールにこのあたりの記述の追加が必要なのかどうか。。。

@hanachin

Copy link
Copy Markdown
Member

LGTMです ✨
文字化けは環境によるのかもしれないですね、GitHubで @higumachan の追加した記事(front matterたぶん当たってない)みても出ない。
http://rhgreadingokinawa.github.io/2014/05/03/event-testpost/

@yamanetoshi

Copy link
Copy Markdown
Contributor Author

これ、わしも忘れてる位なのでどっかに控えた方が良いですね。。

@hanachin

Copy link
Copy Markdown
Member

💭 書かなくてもデフォルトのレイアウト当てるような方法ってないのかな

@hanachin

Copy link
Copy Markdown
Member

_config.ymlで設定すると、書かなくてもlayout自動的にあたるかも
http://jekyllrb.com/docs/configuration/#frontmatter-defaults

@yamanetoshi

Copy link
Copy Markdown
Contributor Author

確認させて頂きます

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