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

홈 화면 - 추가 버튼 개발 #116

Merged
merged 8 commits into from
May 7, 2022
Merged

홈 화면 - 추가 버튼 개발 #116

merged 8 commits into from
May 7, 2022

Conversation

hyesungoh
Copy link
Member

⛳️작업 내용

  • 홈 화면의 Append Button을 추가했어요 (framer상 floating button)

  • 눌렸을 때 나오는 Tooltip을 개발했어요

    • InternalLink에서 사용되는 RouterPathType을 추가하여 적용했어요
  • Image, Link 아이콘을 추가했어요

  • reset css > a 태그에 cursor: pointer 옵션을 추가했어요

📸스크린샷

스크린샷 2022-05-06 오전 2 21 43

⚡️확장 방향

현재 쓰이고 있는 PortalWrapper를 확장하여 backdrop과 함께 position: fixed인 wrapper를 개발해보면 좋을 거 같다고 느꼈어요!

🔥 이슈 공유

#114 에서 확인하실 수 있듯이, forwardRef 경고를 받았었는데, 링크된 아티클에서도 말하듯 html 태그로 감싸주어 현재는 경고를 발생시키지 않네요!! 나중에 필요해질 때 적용해보면 좋을 거 같아용

<InternalLink href={href}>
  <a css={anchorCss}>
     {icon}
     <span>{title}</span      
  </a>
</InternalLink>

@hyesungoh hyesungoh added feat 새로운 기능 추가 style CSS 및 UI 디자인 변경 labels May 5, 2022
@hyesungoh hyesungoh added this to the Sprint 2 milestone May 5, 2022
@hyesungoh hyesungoh added this to In progress in v1.0 via automation May 5, 2022
@hyesungoh hyesungoh self-assigned this May 5, 2022
@hyesungoh hyesungoh linked an issue May 5, 2022 that may be closed by this pull request
3 tasks
@codecov
Copy link

codecov bot commented May 5, 2022

Codecov Report

Merging #116 (895fd53) into main (d881f84) will increase coverage by 0.01%.
The diff coverage is 16.21%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #116      +/-   ##
==========================================
+ Coverage   16.28%   16.29%   +0.01%     
==========================================
  Files         102      106       +4     
  Lines         866      902      +36     
  Branches      130      131       +1     
==========================================
+ Hits          141      147       +6     
- Misses        701      731      +30     
  Partials       24       24              
Impacted Files Coverage Δ
src/components/common/PortalWrapper.tsx 0.00% <0.00%> (ø)
src/components/home/AppendButton.tsx 0.00% <0.00%> (ø)
src/components/home/AppendTooltip.tsx 0.00% <0.00%> (ø)
src/constants/motions.ts 0.00% <0.00%> (ø)
src/hooks/common/useInternalRouter.ts 75.00% <ø> (ø)
src/pages/index.tsx 0.00% <0.00%> (ø)
src/styles/GlobalStyle/reset.ts 0.00% <ø> (ø)
src/components/common/icons/ImageIcon.tsx 66.66% <66.66%> (ø)
src/components/common/icons/LinkIcon.tsx 66.66% <66.66%> (ø)
src/components/common/icons/index.ts 100.00% <100.00%> (ø)

Copy link
Collaborator

@ddarkr ddarkr left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~!!! 👍

image

메인 페이지 그리드가 width height가 지정되어 있어서 그런지 모르겠는데, 이상하게 제 환경에서는 깨지네요... 메인 페이지 그리드 스타일링 풀면 해결되는 것으로 보아 구현해주신 것 문제는 없고 메인 페이지 스타일을 조금 수정해봐야 할 것 같아요!

@hyesungoh
Copy link
Member Author

그런지 모르겠는데, 이상하게 제 환경에서는 깨지네요... 메인 페이지 그리드 스타일링 풀면 해결되는 것으로 보아 구현해주신 것 문제는 없고 메인 페이지 스타일을 조금 수정해봐야 할

맞아요!! 메인 페이지 스타일링은 이거 전에 날린 PR에서 해결되었어요!! 머지 후에도 테스트 해보겠슴다 👍 👍

@ddarkr
Copy link
Collaborator

ddarkr commented May 5, 2022

@hyesungoh 오 이미 해결된 부분이였군요...!!! 죄송합니다 😂

굳이에요~!! 👍

@cloudflare-pages
Copy link

cloudflare-pages bot commented May 6, 2022

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 895fd53
Status: ✅  Deploy successful!
Preview URL: https://a43e515a.ygt.pages.dev

View logs

@hyesungoh hyesungoh linked an issue May 6, 2022 that may be closed by this pull request
2 tasks
@hyesungoh hyesungoh removed a link to an issue May 6, 2022
2 tasks
Copy link
Collaborator

@sensecodevalue sensecodevalue left a comment

Choose a reason for hiding this comment

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

홀리 완벽합니다!

@@ -2,7 +2,7 @@ import { useMemo } from 'react';
import { useRouter } from 'next/router';
import { UrlObject } from 'url';

export type RouterPathType = `/` | '/test';
export type RouterPathType = `/` | '/test' | '/add/link' | '/add/text' | '/add/image';
Copy link
Member

Choose a reason for hiding this comment

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

아하!! 감사합니다 :)

@positiveko
Copy link
Member

고생하셨습니다 :) ⚡️⚡️⚡️⚡️⚡️⚡️⚡️⚡️

@hyesungoh hyesungoh merged commit d41b221 into main May 7, 2022
v1.0 automation moved this from In progress to Done May 7, 2022
@positiveko positiveko deleted the issue/112 branch May 12, 2022 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 새로운 기능 추가 style CSS 및 UI 디자인 변경
Projects
v1.0
Done
Development

Successfully merging this pull request may close these issues.

홈 화면 - 추가 버튼
4 participants