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

Kdt5_park jin young #76

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Kdt5_park jin young #76

wants to merge 8 commits into from

Conversation

jinyoungpark231
Copy link

클론코딩 과제


설명

도미노피자 광고페이지 이미지에서
메인 , 이벤트 , 프로모션페이지를 연동해
클릭시 클릭한 해당페이지로 갈 수 있게 했습니다.

Copy link
Member

@ParkYoungWoong ParkYoungWoong left a comment

Choose a reason for hiding this comment

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

최종 의견은 XXX.. 이렇습니다~

}

/* main */
.main{
Copy link
Member

Choose a reason for hiding this comment

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

main HTML 요소가 있는데, 별도의 클래스를 가진 요소로 만드셨네요.
이유가 뭘까요?

<link rel="stylesheet" href="./css/main.css"/>
</head>
<body>
<div class="main">
Copy link
Member

Choose a reason for hiding this comment

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

그냥 <main> 으로 만드시면 되지 않을까요?

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