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] iOS에서 아이콘이 제대로 보이지 않던 문제 #948

Merged
merged 2 commits into from
Jan 15, 2025

Conversation

Todari
Copy link
Contributor

@Todari Todari commented Jan 15, 2025

issue

구현 목적

현재 iOS에서 svg아이콘이 제대로 보이지 않는 문제가 있습니다.
이를 바로 잡기 위해 IconComponent를 변경합니다.

구현 사항

�svg의 크기를 동적으로 조절하기 위해

export const svgStyle = css`
  svg {
    width: auto;
    height: auto;
  }
`;

처럼 사용했지만, 크롬에서는 부모의 위치에 맞도록 설정되지만, safari에선 fixed로 되는 것이 기본 설정이기 때문에, os에 따라서 다르게 보이는 문제가 있었습니다.

이에 아래와 같이 변경하였습니다.

export const svgStyle = css`
  width: 100%;
  height: 100%;

  svg {
    width: 100%;
    height: 100%;
  }
`;

중점적으로 리뷰받고 싶은 부분(선택)

어떤 부분을 중점으로 리뷰했으면 좋겠는지 작성해주세요.

논의하고 싶은 부분(선택)

논의하고 싶은 부분이 있다면 작성해주세요.

🫡 참고사항

@Todari Todari added 🖥️ FE Frontend 🚨 bug bug labels Jan 15, 2025
@Todari Todari added this to the v3.1.3 milestone Jan 15, 2025
@Todari Todari requested review from pakxe, soi-ha and jinhokim98 January 15, 2025 09:24
@Todari Todari self-assigned this Jan 15, 2025
Copy link
Contributor

@soi-ha soi-ha 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

Copy link

Copy link
Contributor

@jinhokim98 jinhokim98 left a comment

Choose a reason for hiding this comment

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

good 제발~~

@Todari Todari merged commit 087657e into fe-dev Jan 15, 2025
2 checks passed
@Todari Todari deleted the feature/#947 branch January 15, 2025 09:47
This was referenced Jan 15, 2025
Todari added a commit that referenced this pull request Jan 15, 2025
* fix: Icon component의 viewBox px 단위로 변경

* fix: Icon Component가 제대로 보이지 않던 문제 해결
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants