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

style: Profile 컴포넌트 UI #13

Merged
merged 2 commits into from
Jul 20, 2022
Merged

style: Profile 컴포넌트 UI #13

merged 2 commits into from
Jul 20, 2022

Conversation

paya17
Copy link
Collaborator

@paya17 paya17 commented Jul 20, 2022

Profile 컴포넌트 UI 구현
image

@paya17 paya17 linked an issue Jul 20, 2022 that may be closed by this pull request
5 tasks
@paya17 paya17 removed a link to an issue Jul 20, 2022
5 tasks
@paya17 paya17 linked an issue Jul 20, 2022 that may be closed by this pull request
6 tasks
Copy link
Collaborator

@flowersayo flowersayo left a comment

Choose a reason for hiding this comment

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

너무너무 수고 많으셨습니다!!!! 🤗💕


</ProfileBox>

</>
Copy link
Collaborator

Choose a reason for hiding this comment

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

<> </> 부분은 삭제하는 거 어떤가용?!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

헛 필요없겠네요!! 코멘트 감사해요 서연님!!🥰

height: 575rem;
background-color: #FFFBF2;
border-top-left-radius: 30rem;
border: 1rem solid #D3CEC4;
Copy link
Collaborator

Choose a reason for hiding this comment

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

색상부분도 var(-- ) 로 가져다 쓰면 좋을 것 같습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

앗 넵넵! 아무생각없이 만들었네요 하하핳🙈

* {
box-sizing: border-box;
}

Copy link
Collaborator

Choose a reason for hiding this comment

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

센스쟁이,,,, 👍

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

앗 아녜욥ㅎㅎㅎ 서연님이 저보다 훨씬 더 센스쟁이쥬,,,😎

프로필 수정
</ProfileEditBtn>
</BtnBox>
{/* {showModal && <ProfileEditModal imageInfo={info.image} bioInfo={info.bio} isOpenModal={showModal} setIsOpenModal={setShowModal}/>} */}
Copy link
Collaborator

Choose a reason for hiding this comment

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

혹시 이거 react-modal 사용하셔서 구현하신건가용?!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

모달은 아직 만드는 중인데 react-modal은 사용하지 않고 그냥 코드 작성해서 만들고 있어요!(트위터 클론코딩때 썼던 코드 가져와서요ㅎㅎ)

@@ -1,15 +1,143 @@
import React from "react";
import React, {useState, useEffect} from 'react';
import DefaultImg from "../../assets/DefaultImg.png";
Copy link
Collaborator

Choose a reason for hiding this comment

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

저도 이번에 알았는데 png보다 svg 파일로 사용하면 확대했을때 화질이 덜깨지는 것 같더라구용,,,!
피그마에서 svg 형태로도 받을 수 있게끔 되있어서 svg 로 사용해봐도 좋을 듯 합니다😆

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

아 그렇군요!! 정보 감사합니다❤️

@flowersayo flowersayo merged commit a8fafd4 into main Jul 20, 2022
@flowersayo flowersayo deleted the feature/Profile branch July 22, 2022 13:50
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