Skip to content

pc版UIサンプル作成#15

Open
yusuke-creater wants to merge 2 commits into
mainfrom
pc_UI-yusuke
Open

pc版UIサンプル作成#15
yusuke-creater wants to merge 2 commits into
mainfrom
pc_UI-yusuke

Conversation

@yusuke-creater
Copy link
Copy Markdown
Collaborator

簡易的ということで・・・
改善点あれば教えて頂きたいです。
確認お願いします!

@96-38
Copy link
Copy Markdown
Collaborator

96-38 commented Dec 14, 2020

UI作成ありがとうございます!ヘッダー部分の見た目はデザインに寄せられていると思います!
見た目に関係するPRを出す時は下記のようにスクリーンショットをいれると、より見やすいPRになると思います👍
スクリーンショットを見てもらうと分かりますが、画像が表示されていないです...!画像ファイルを push できているか一度確認お願いします!

PC向け仮UI

Comment thread sample/pc_UI/index.css
@@ -0,0 +1,51 @@
/* @charset "UTF-8"; */
body{
margin: 0 0 0 0;
Copy link
Copy Markdown
Collaborator

@96-38 96-38 Dec 14, 2020

Choose a reason for hiding this comment

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

margin-top, margin-right, margin-bottom, margin-left が同じ値の場合は
margin : 0 ;
と記述すればOKです!

Comment thread sample/pc_UI/index.css
margin: 20px 70px 20px 50px;
}
.photo2 {
margin: 20px 0 20px 0;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

margin-top と margin-bottom が、margin-right と margin-left がそれぞれ同じ値の場合は
margin : 20px 0px
と記述すればOKです!

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