[Fix] 최종 QA 반영 #281
Merged
[Fix] 최종 QA 반영 #281
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Deploying sopt-recruiting-frontend with
|
| Latest commit: |
fc27ac2
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://11bb7233.sopt-recruiting-frontend.pages.dev |
| Branch Preview URL: | https://fix--280-qa.sopt-recruiting-frontend.pages.dev |
eonseok-jeon
approved these changes
Jul 30, 2024
Comment on lines
170
to
172
| <p className={success}>이메일이 전송되었어요. 네트워크로 인해 약 1분 정도 소요될 수 있어요.</p> | ||
| <p className={success} style={{ marginTop: '-8px' }}> | ||
| 네트워크 상황에 따라 약 1분 정도 소요될 수 있어요. | ||
| 약 1분 이후 메일이 오지 않는다면 스팸 메일함을 확인해주세요. |
There was a problem hiding this comment.
글자가 너무 많아진 거 같아요
네트워크로 인해 약 1분 정도 -> 네트워크 지연 등이 아닌 네트워크 자체는 원인이 아닌 거 같아요
약 1분 이후 메일이 오지 않는다면 -> 위의 약 1분 정도와 문구가 겹치는 거 같아요
따라서
이메일이 전송되었어요. 약 1분 정도 소요될 수 있어요.
메일이 오지 않는다면 스팸 메일함을 확인해주세요.
로 변경하는 거 어떠신가요?
There was a problem hiding this comment.
이거 디자인쪽에서 정해주신 거라서 슬랙으로 논의 옮길게요!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related Issue : Closes #280
🧑🎤 Summary
🧑🎤 Comment
필수항목 점 삐져나오는 이슈는
예전에도 언석오빠가 border이 살짝 보인다고 border을 흰색처리 하는 식으로 해결했던 것 같은데,
그 ApplyHeader의 여백이 padding이 아니라 margin으로 부여되고 있어서 배경색이 양옆으로는 적용이 안되고 있더라구요
즉 저 초록색 영역이 지금은 padding으로 줘서 저렇게 초록이지만, 다 주황색(margin)이었어요!
그래서 background color가 내부 파란색 부분에만 적용되고 있어서 그 영역으로 조금이라도 벗어나는 애들은 다 비쳤던겁니다
padding으로 바꿔서 해결해줬어요~~