Feat/map #10
Merged
Feat/map #10
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
페이지 이름 request 괜찮은가요?
SearchBar가 jusetify-content: center를 안해도 양쪽으로 뻗는데 아직 이유를 모르겠어요.
stack이 스크린 화면 100%로 꽉 찼으면 좋겠는데 계속 넘쳐서 애먹고 있어요.
|
Hong-been
reviewed
Apr 30, 2022
Hong-been
reviewed
Apr 30, 2022
hotbreakb
commented
May 1, 2022
| > | ||
| <HeaderBar /> | ||
| <SearchBar /> | ||
| <div css={{flex: 1, height: 10}}> |
There was a problem hiding this comment.
이 부분 해결했습니다!
해결방식
- stack을 감싸는 div에 height를 준다.
- 홍빈님 말씀대로 해봤는데 아직 의문인 것은 height를 몇으로 주든 원하는 대로 동작은 되는데
height: 10px로 고정되지는 않는다는 것입니다.
- stack 안에 들어가는 게시글 하나에 div를 만든다.
- 따로 감쌀 필요가 없을 거 같아서
<></>로 했더니 안 되더군요.
| css={{ | ||
| display: "flex", | ||
| flexDirection: "column", | ||
| width: "100%", |
There was a problem hiding this comment.
홍빈님 말씀대로 이 부분의 영향이 있었어요. 확인해보니, SearchBar 안에 들어있는 컴포넌트의 margin, padding을 설정해주지 않아서 마음대로 움직이고 있는 거였어요.
ghtea
reviewed
May 1, 2022
| onChange?: ChangeEventHandler<HTMLSelectElement>; | ||
| }; | ||
|
|
||
| export const Select = ({ |
There was a problem hiding this comment.
색상 값이 사실 컬러시스템에서 가져오지 않고 hex 값을 사용하는 등
디자인팀에서 아직 와이어프레임 단계이다보니깐 아직 디자인이 미완성인거 같아요.
지금은 당장 이렇게 나두고 나중에 디자인 팀이 컴포넌트로 작업해주시면 그때 수정하는 PR 올릴게요!
ghtea
approved these changes
May 1, 2022
| @@ -31,6 +31,7 @@ | |||
| "@storybook/react": "^6.4.22", | |||
| "@storybook/testing-library": "^0.0.9", | |||
| "@svgr/webpack": "5.5.0", | |||
| "@types/navermaps": "^3.0.13", | |||
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
헌혈 요청 페이지 만드는 중
Key Changes
Notes
SearchBar가jusetify-content: center를 안해도 양쪽으로 뻗는데 아직 이유를 모르겠어요.stack이 스크린 화면 100%로 꽉 찼으면 좋겠는데 계속 넘쳐서 애먹고 있어요.Screenshots
Checklist
testlint-fixts-check