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] '약속 후보 날짜 선택'과 '약속 시간 범위 선택' 사이에 간격을 설정 #397

Merged
merged 4 commits into from
Oct 11, 2024

Conversation

Yoonkyoungme
Copy link
Contributor

@Yoonkyoungme Yoonkyoungme commented Oct 10, 2024

관련 이슈

작업 내용

1. '약속 후보 날짜 선택'과 '약속 시간 범위 선택' 사이에 간격을 설정

AS-IS

기존 코드는 아래 사진처럼 '약속 후보 날짜 선택'과 '약속 시간 범위 선택' 사이에 간격이 설정되어 있지 않습니다.
image

TO-BE

아래와 같이 간격을 지정하였습니다.
image


2. iOS에서 logoSunglass 배경이 검은색으로 설정되는 문제 해결

AS-IS

기존에는 아래 사진처럼 iOS에서 logoSunglass svg의 배경색이 검은색으로 표시되는 문제가 있었습니다.
image

TO-BE

svg에서 <pattern> 요소가 제대로 설정되지 않거나, 참조되는 <use> 요소에 문제가 있을 경우, iOS는 빈 패턴 대신 기본 배경색으로 검은색을 사용합니다.

svg 패턴을 정의하지 않으면 iOS에서 기본 배경색으로 검정이 표시되는 것이 문제의 원인이었어요🤣 아래 코드처럼 pattern을 설정해 주었습니다

<defs>
  <pattern id="pattern0_2561_1931" patternContentUnits="objectBoundingBox" width="1" height="1">
    <use xlink:href="#image0_2561_1931" />
  </pattern>
</defs>

해리 휴대폰으로 iOS 기기에서 배경색이 더 이상 검은색이 아닌 것을 확인했습니다!

특이 사항

첫 번째 커밋 rename: CreateMeetingPage 디렉터리 내부 구조 변경은 다른 pages 디렉터리 형식과 통일 시키기 위한 작업입니다.(MeetingDateTime, MeetingHostInfo, MeetingName 컴포넌트를 components 디렉터리 내부로 이동)
이 부분은 코드 변경이 없으니 리뷰 하지 않아도 될 것 같아요!

리뷰 요구사항 (선택)

@Yoonkyoungme Yoonkyoungme added 🎨 디자인 디자인을 해요 :) ♻️ 리팩터링 코드를 깎아요 :) labels Oct 10, 2024
@Yoonkyoungme Yoonkyoungme self-assigned this Oct 10, 2024
Copy link

github-actions bot commented Oct 10, 2024

Test Results

9 tests   9 ✅  11s ⏱️
2 suites  0 💤
1 files    0 ❌

Results for commit 100d5b9.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@hwinkr hwinkr left a comment

Choose a reason for hiding this comment

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

고생했어요 빙봉!🔮

아이러니하게도 빙봉은 갤럭시 유저인데 IOS 대응을 제일 많이하는 것 같네요...ㅋㅋㅋ

Copy link
Contributor

@Largopie Largopie left a comment

Choose a reason for hiding this comment

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

👍👍👍💯💯💯

@Yoonkyoungme Yoonkyoungme merged commit 1ea14a4 into develop Oct 11, 2024
5 checks passed
@Yoonkyoungme Yoonkyoungme deleted the design/396-apply-gap branch October 11, 2024 17:14
@Yoonkyoungme Yoonkyoungme added the 🐈 프론트엔드 프론트엔드 관련 이슈에요 :) label Oct 17, 2024
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.

[FE] '약속 후보 날짜 선택'과 '약속 시간 범위 선택' 사이에 간격을 설정해요 :)
3 participants