Skip to content

Conversation

kotuakoutyan
Copy link
Contributor

  • 記事の投稿方法と方針 を読んだ
  • ローカルで見た目を確認した
  • lintで文章をチェックした
  • default.html の変更箇所にコメントをいれた

やったこと

  • LTのスライドを上げた

@curegit
Copy link
Contributor

curegit commented May 23, 2019

  • ローカルで見た目を確認した
  • lintで文章をチェックした

@curegit curegit added the new post 📨 新しい記事の投稿 label May 23, 2019
@curegit curegit requested review from hatosh, rilu0361 and curegit May 23, 2019 15:07

#### Unity 入門 -2019-

<iframe src="https://docs.google.com/presentation/d/e/2PACX-1vR7lVkBVJCGLXYu-4RlkshHAdLCrxftDeTCWQ2TZb2tDXOb8spNhBEsKC0O5OUBqiB8WhNdEK8O1kpo/embed?start=false&loop=false&delayms=3000" frameborder="0" width="960" height="569" allowfullscreen="true" mozallowfullscreen="true" webkitallowfullscreen="true"></iframe>
Copy link
Collaborator

Choose a reason for hiding this comment

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

これまでは width="480" height="299" だったので合わせたほうがいいのでは?

Copy link
Contributor

Choose a reason for hiding this comment

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

過去分も含めて直そうと思います

Copy link
Collaborator

@ahuglajbclajep ahuglajbclajep May 24, 2019

Choose a reason for hiding this comment

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

width="640" height="390" でどうですか?
width は、本文の横幅を決めてる inner クラスが max-width: 640px; となっているので、これで横いっぱいに表示できる。
https://github.com/prog-g/prog-g.github.io/blob/361a51c/_layouts/default.html#L47
https://github.com/pages-themes/slate/blob/v0.1.1/_sass/jekyll-theme-slate.scss#L264

スライドは 16:9 で作るとして、640/16*9=360 に操作バーの部分を足して height は 390 でレターボックスが目立たずいい感じになりました。

Copy link
Contributor

Choose a reason for hiding this comment

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

ではこのPRはこのままマージして、あとから全部まとめて 640x390 にするので良いですか?

@curegit curegit merged commit bffbc46 into prog-g:master May 26, 2019
@kotuakoutyan kotuakoutyan deleted the NewPost branch June 2, 2019 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new post 📨 新しい記事の投稿
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants