-
Notifications
You must be signed in to change notification settings - Fork 0
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
7차 세미나 구현 및 이미지 업로드 구현 #5
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰 남기고 갑니다 총총,,
마지막까지 넘 고생많았어!!!!
import React from "react"; | ||
import styled from "styled-components"; | ||
|
||
const ImagWrapper = ({ ratio, children }) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Imag나 Img 둘중 하나로 통일하면 편할 듯 합니닷!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Visual Studio Code에서는 변수 선택하고 F2 누르면 파일 곳곳에 있는 변수 이름들을 한방에 바꿀 수 있어요~
setArticleData(tempArticleData); | ||
}; | ||
|
||
const handleImageChange = async (e) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PublishScreen/PublishLeftScreen.jsx
에 있는 함수랑 내용과 기능이 같네요!!
이 부분을 따로 뺀 뒤 import해서 사용해도 좋을 것 같아요!!
📌 내용
기본 과제를 구현하였습니다!
기본과제
📌 PR Point
📌 스크린샷