[Feat] 차트 레이아웃 시도 (svg, canvas) #12
Merged
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
HongJungKim-dev
approved these changes
May 27, 2022
FE/src/App.tsx
Outdated
| @@ -7,6 +9,8 @@ function App() { | |||
| <div> | |||
| <button onClick={handleOpenModal}>로그인 모달</button> | |||
| <LoginModal isModalOpen={isModalOpen} handleOpenModal={handleOpenModal} /> | |||
There was a problem hiding this comment.
이 부분은 지워주셔도 될것 같습니다!
| return ( | ||
| <div style={container}> | ||
| <svg viewBox={`0 0 100 100`} style={{ border: '3px solid #ccc' }}> | ||
| <polyline fill="none" stroke="tomato" strokeWidth={1} points={points} /> |
There was a problem hiding this comment.
strokeWidth는 단위를 안써도 될까요?
There was a problem hiding this comment.
이 부분은 저도 이해가 부족한 것 같아서 추가적으로 학습해보겠습니다!
HongJungKim-dev
changed the title
HongJungKim-dev
added a commit
that referenced
this pull request
May 30, 2022
* [Feat] 모달 구현 (#8) * docs: 템플릿 수정(리뷰반영) * refactor: config폴더내의 파일을 루트로 이동(리뷰반영) * build: react-refresh 추가 * feat: 모달 구현 * refactor: config폴더내의 파일을 루트로 이동(리뷰반영) * build: react-refresh 추가 * build: 빌드 수정 * build: webpack styled options 수정 * [feat] 캘린더 한 달 뷰 (#11) * feat: 현재 달의 calendar를 모달에 출력 * fix: calendar날짜가 마지막 일까지 출력되도록 수정 * refactor: 캘린더 뷰에 사용하는 상수를 파일로 분리 * [feat] 차트 레이아웃 시도 (svg, canvas) (#12) * Feat: svg 그래프 * Feat: canvas 레이아웃 - 선 그리는 로직 미완 * Chore: svgChart에서 plots 좌표값 관련 주석 작성 Co-authored-by: HongJungKim(이든) <58525009+HongJungKim-dev@users.noreply.github.com> Co-authored-by: khj <hongjung.kim.dev@gmail.com>
kimyouknow
added a commit
that referenced
this pull request
Jun 1, 2022
* [Feat] 모달 구현 (#8) * docs: 템플릿 수정(리뷰반영) * refactor: config폴더내의 파일을 루트로 이동(리뷰반영) * build: react-refresh 추가 * feat: 모달 구현 * refactor: config폴더내의 파일을 루트로 이동(리뷰반영) * build: react-refresh 추가 * build: 빌드 수정 * build: webpack styled options 수정 * [feat] 캘린더 한 달 뷰 (#11) * feat: 현재 달의 calendar를 모달에 출력 * fix: calendar날짜가 마지막 일까지 출력되도록 수정 * refactor: 캘린더 뷰에 사용하는 상수를 파일로 분리 * [feat] 차트 레이아웃 시도 (svg, canvas) (#12) * Feat: svg 그래프 * Feat: canvas 레이아웃 - 선 그리는 로직 미완 * Chore: svgChart에서 plots 좌표값 관련 주석 작성 * build: react-refresh 추가 * [PR] 1주차 두번째 PR 리뷰 반영 (#14) * refactor: 관심사 분리를 고려하여 모달 리팩토링 응집성을 위하여 html에서 모달 태그 삭제, Portal 컴포넌트에서 해당 element 생성 * refactor: interface 네이밍 수정 컴포넌트 이름 + Props (ex.ComponentProps) * build: liveReload 기능 false처리 * [Feat] 캘린더 렌더링 로직 구현 (#17) * Feat: 캘린더 렌더링 * Chore: 캘린더 관련 주석 작성 * [Feat] 차트 그래프 기본 구현 (#18) * feat: 가격 범위 차트 그래프 렌더링 * refactor: 중복 스타일 mixin으로 분리 중복된 캔버스 컨테이너 스타일 분리 Co-authored-by: HongJungKim(이든) <58525009+HongJungKim-dev@users.noreply.github.com> Co-authored-by: khj <hongjung.kim.dev@gmail.com>
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.
About
svg와 cavnas로 꺾은선 차트를 그리는 레이아웃을 시도했습니다.
Description
미완)추가사항
canvas관련해서 추가적인 학습이 필요합니다.
Result
Ref
#10