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

feat(view): ClusterGraph.const.ts의 DETAIL_HEIGHT 값 수정 #382

Merged
merged 1 commit into from
Aug 13, 2023

Conversation

kimdonggu42
Copy link
Member

Related issue

resolve #347

Result

first_PR

Work list

Summary : DETAIL_HEIGHT을 245로 수정

Detail : clusterGraph.hook의 135번째 줄에서 drawClusterGraph가 DETAIL_HEIGHT을 인자로 받고 있는데 이 값이 ClusterGraph.const에서 220으로 지정되어 있었습니다.
그런데 summary의 detail_container에는 height 220px에 추가로 위, 아래 padding 값이 5px, 20px로 지정되어 있어 detail 오픈 시 높이가 어긋나는 현상이 있던 것으로 확인했습니다.

Discussion

@kimdonggu42 kimdonggu42 requested review from a team as code owners August 10, 2023 14:00
Copy link
Contributor

@ytaek ytaek left a comment

Choose a reason for hiding this comment

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

움짤로 올려주셔서 감사합니다!! : -)

Copy link
Contributor

@rlaqltmxm rlaqltmxm left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@kimdonggu42 kimdonggu42 merged commit 0b6947c into githru:main Aug 13, 2023
2 checks passed
@2taesung
Copy link
Contributor

2taesung commented Aug 17, 2023

수고하셨습니다! 디테일한 문제 잡아주셨네요!!!

@kimdonggu42 kimdonggu42 self-assigned this Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[view] ClusterGraph의 위치가 Summary에서 Detail이 펼쳐질 때 어긋나는 현상 해결하기
4 participants