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

[FE] 차트 고도화 & favicon & 화면 레이아웃 일부 수정. #228

Merged
merged 9 commits into from
Nov 28, 2024

Conversation

dannysir
Copy link
Collaborator

두 가지 PR 템플릿 중 하나를 골라 작성하시면 됩니다!

✅ 주요 작업

  • 마우스 위치에 따른 값을 좌측 상단에 표시 (차트 고도화)
  • favicon 설정.
  • 로고 이미지 포멧 수정.
  • 이동 평균선 on/off 기능 추가.
스크린샷 2024-11-28 오후 2 07 38

💭 고민과 해결과정

  • 고민과 해결과정

📊 FE/BE 전체 작업 내역

@dannysir dannysir added FE 프론트엔드 LAYOUT 레이아웃 구현 labels Nov 28, 2024
@dannysir dannysir requested a review from dongree November 28, 2024 05:08
@dannysir dannysir self-assigned this Nov 28, 2024
Copy link
Collaborator

@dongree dongree left a comment

Choose a reason for hiding this comment

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

이동평균선과 시가, 종가 등 정보들을 왼쪽 상단에 추가해주니까 더 보기 좋네요 👍👍
수고하셨습니다!!

padding,
0.1,
);
if (moveAverageToggle) {
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.

이동 평균선 지웠다가 넣었다가하는 용도로 사용했습니다.

@@ -2,7 +2,7 @@
<html lang="en">
<head>
<meta charset="UTF-8" />
<link rel="icon" type="image/svg+xml" href="/vite.svg" />
<link rel="icon" href="src/assets/favicon.ico" type="image/x-icon" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

type="image/x-icon" 은 무엇을 의미하나요??

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

vite에서 파비콘 설정할 때 아이콘 타입까지 넣어줄 용도로 넣었습니다.

@dongree dongree merged commit 576c292 into front/main Nov 28, 2024
2 checks passed
@dannysir dannysir deleted the feature/layout/detail/chart-#192 branch December 1, 2024 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 LAYOUT 레이아웃 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants