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

14-9 [FE] [논문 상세 - 네트워크 차트] 그래프 사용 안내 tooltip을 표시한다. #119

Merged
merged 8 commits into from
Dec 12, 2022

Conversation

Palwol
Copy link
Collaborator

@Palwol Palwol commented Dec 12, 2022

개요

  • 논문 상세 페이지 진입 시 그래프 사용 안내가 적힌 툴팁을 표시합니다.
  • 툴팁은 여닫을 수 있으며, 상세 페이지 최초 진입 시 툴팁이 열린 상태로 표시됩니다.
  • 툴팁을 한 번 닫은 이후에는 다시 상세 페이지에 진입했을 때 툴팁이 닫힌 상태로 표시됩니다.

image

작업사항

  • 논문 상세 페이지 그래프 사용 안내 툴팁 구현
  • 툴팁을 한 번 닫으면 sessionStorage에 isTooltipclosed key를 true로 저장
  • isTooltipclosed이 true면 논문 상세 페이지 진입 시 툴팁이 닫힌 상태로 보여짐
  • 검색목록에서 저자 이름이 아래로 내려오는 현상 수정, 저자 길이 한 줄로 수정

@@ -2,7 +2,8 @@ import styled from 'styled-components';

interface IProps {
icon: JSX.Element;
onClick: React.MouseEventHandler;
onClick?: React.MouseEventHandler;
'aria-label': string;
Copy link
Member

Choose a reason for hiding this comment

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

컴포넌트 특성상 aria-label을 필수 타입으로 명시해두는 것도 좋네요.

frontend/src/icons/XMark.tsx Outdated Show resolved Hide resolved
const item = window.sessionStorage.getItem(key);
if (item) return JSON.parse(item);
return null;
};
Copy link
Member

Choose a reason for hiding this comment

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

스토리지를 한 파일에서 관리하는것도 좋네요~

Co-authored-by: yeynii <yeynii@gmail.com>
Copy link
Collaborator

@JunYupK JunYupK left a comment

Choose a reason for hiding this comment

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

확인했습니다. 수고하셨습니다~

@Palwol Palwol merged commit 5b4aee3 into dev Dec 12, 2022
@Palwol Palwol deleted the feature/graph-info branch December 12, 2022 15:52
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.

3 participants