Skip to content

homework1- David(Chen)#22

Open
magicXue wants to merge 2 commits intodistinctioncoding:mainfrom
magicXue:main
Open

homework1- David(Chen)#22
magicXue wants to merge 2 commits intodistinctioncoding:mainfrom
magicXue:main

Conversation

@magicXue
Copy link

No description provided.

Copy link

@Gary-Distinctioncoding Gary-Distinctioncoding left a comment

Choose a reason for hiding this comment

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

命名规范:使用kebab-case,eg. .left_panel -> .left-panel
注意%的unit, 是根据的父元素的宽高的百分之多少进行计算,前提是父元素要有合理的宽高

Comment on lines +20 to +28
.signup_form {
height: 36vh;
width: 25vw;
max-width: 350px;
background-color: white;
padding: 10px 30px;
box-shadow: 0 4px 15px rgba(0, 0, 0, 0.3);
border-radius: 8px;
}

Choose a reason for hiding this comment

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

should not set a height, the conent in this element will cause overflow
image

<div class="main_login">
<div class="left_panel">
<div class="welcome_message">
<H2 class="welcome_title">Welcome to Paint Quote System</H2>

Choose a reason for hiding this comment

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

head tag should be small case h2

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