Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
hwinkr created a review comment on a pull request on woowacourse-teams/2024-momo
그게 훨씬 좋고 더 직관적인 것 같아요! 감사합니다 ㅎ
hwinkr created a review comment on a pull request on woowacourse-teams/2024-momo
@hw0603 페드로와 함께 달력 UI에 대해서 QA를 진행했었는데요..! 0.02rem이 사각형과 삼각형 사이의 간격을 최소화할 수 있는 위치인 것 같아서 이렇게 수정하게 되었습니다!
hwinkr created a review comment on a pull request on woowacourse-teams/2024-momo
이거 머지되면 같이 충돌해결해봐야 할 것 같아요...!
hwinkr created a review comment on a pull request on woowacourse-teams/2024-momo
`:focus` 가상 클래스 선택자는 `<input />` 태그가 포커스 되었을 때 사용할 수 있는 클래스 선택자입니다! 현재 해당 스타일 로직은 인풋의 스타일을 결정하기 위한 것이 아닌 라벨의 텍스트 색상을 적용하기 위한 로직입니다! ```css export const s_floatingLabelContainer = css` posit...
hwinkr created a review comment on a pull request on woowacourse-teams/2024-momo
> _이름을 floating으로 지으신 이유가 궁금하네요❗️_ 보통 `<label />`은 `<input />`위에 위치하게 되는데 이번 PR에서 생성한 UI의 경우 관습적인 위치에 있는 것이 아니라 `<input />` 내부에 위치하게 됩니다. 그래서 떠다닌다는 느낌을 명시해 주기 위해서 해당 이름을 선택하게 되었습니다. 처음에는 `Floati...
hwinkr created a review comment on a pull request on woowacourse-teams/2024-momo
아하 apperance 속성을 부여하면 색상도 사라지는 군요...! 드롭다운 문제도 같이 해결하도록 할게요.
Yoonkyoungme created a review on a pull request on woowacourse-teams/2024-momo
[P1] 해리 실패하는 테스트가 있네요. 확인해 주세요! <img width="952" alt="image" src="https://github.com/user-attachments/assets/3d653d70-a3f6-46dc-aa22-e210871a757d">
Yoonkyoungme created a review comment on a pull request on woowacourse-teams/2024-momo
이건 저번 리뷰에서 놓쳤던 부분인데, 갑자기 보이네용🤣 `moveToPrevMonth`와 `moveToNextMonth` 함수에서 `new Date(currentYear, currentMonth ± 1)`를 각각 호출하고 있는데, 이 로직이 중복되는 것 같아요. 중복되는 부분을 아래처럼 별도의 함수로 분리해서 사용해보는 것을 가볍게 제안해 봅니...
Largopie created a review comment on a pull request on woowacourse-teams/2024-momo
[Q] 어째서 0.04rem이 아니라 0.02rem일까요?
Largopie created a review comment on a pull request on woowacourse-teams/2024-momo
[P2] 제가 이전 PR에서 Funnel 마지막 단계인 날짜 선택 부분에서 BottomFixedButton에 가려지는 문제때문에 임시 div 요소를 추가했는데 해당 높이가 6rem으로 설정되어 있습니다. 한번 보고 같이 해결하면 좋을 듯 싶어요! [노션 링크](https://paper-mass-5ff.notion.site/8e55c8102...
Largopie created a review comment on a pull request on woowacourse-teams/2024-momo
[P1] 충돌사항 짱많겠네요 🥹🥹🥹🥹🥹🥹🥹🥹🥹 해당 로직은 uuid를 useContext를 사용하게 함으로써 제거되니 참고해주세요 해리군~ 바로 rebase하고 반영사항도 함께 보면 좋을 것 같습니다!
Largopie created a review comment on a pull request on woowacourse-teams/2024-momo
[Q] 이름을 floating으로 지으신 이유가 궁금하네요❗️ [P2] 현재 placeholder의 색상과 입력 시 색상이 동일합니다. 더 나은 사용자 경험을 위해 입력하는 글자 색상은 검정색으로 유지하는 것이 좋아보여요! <img width="358" alt="image" src="https://github.com/user-at...
Largopie created a review comment on a pull request on woowacourse-teams/2024-momo
[Q] `:focus` 가상 클래스 선택자를 이용하지 않고 `isFocused`를 사용한 이유가 있나요!?
Largopie created a review comment on a pull request on woowacourse-teams/2024-momo
[P3] 지금 당장 반영하는 것은 아니지만 우리 색상을 통일하기 위해서 이런 에러 색상이나 placeholder의 색상을 별도의 상수로 지정하는 것이 좋아보입니다!! 시간날 때 가능한 한 빨리요!
Largopie created a review comment on a pull request on woowacourse-teams/2024-momo
[P2] > https://stackoverflow.com/questions/69743277/how-to-fix-blue-color-of-html-select-tags-in-ios-15-using-css 모바일 화면을 살펴보니 select box도 마찬가지로 글씨가 파란색인 것을 확인했습니다. (제가 물론 커스터마이징을 해야하는 것이지만 ㅠ...
Largopie created a review on a pull request on woowacourse-teams/2024-momo
멋쪄요 해리~! 🥹 페이지 디자인이 갈 수록 멋있어지고 있군요!! 디자인 고민 같이 해주고 반영해주셔서 감사합니다 👍 궁금한 사항이랑 수정 요청사항 함께 리뷰 남겼으니 확인 부탁드려요!
Largopie created a review on a pull request on woowacourse-teams/2024-momo
멋쪄요 해리~! 🥹 페이지 디자인이 갈 수록 멋있어지고 있군요!! 디자인 고민 같이 해주고 반영해주셔서 감사합니다 👍 궁금한 사항이랑 수정 요청사항 함께 리뷰 남겼으니 확인 부탁드려요!
Largopie closed an issue on woowacourse-teams/2024-momo
[FE] 뒤로가기 기능 구현
## 이슈 요약 <!-- 추가하거나 개선하려는 기능에 대해 간결하게 설명해 주세요. --> 헤더를 수정해서 뒤로가기 기능을 구현해요. ## 상세 작업 내용 <!-- - [ ] TODO --> - [ ] 헤더 수정 사항 적용 - [ ] 뒤로가기 기능 구현 ## 레퍼런스 <!-- 이 기능과 관련된 추가적인 정보나 참...
Largopie pushed 1 commit to develop woowacourse-teams/2024-momo
- [FE] 헤더 뒤로가기 기능 구현 (#411) * design: 토스트 컨테이너 패딩 조절 - 토스트 메시지의 padding 속성으로 입력부분이 가려지는 문제가 있어 위와 아래의 패딩을 0으로 설정 ... fa329ba
Largopie closed a pull request on woowacourse-teams/2024-momo
[FE] 헤더 뒤로가기 기능 구현
## 관련 이슈 - resolves: #394 ## 작업 내용 > 사진으로 자세히 보고 싶거나, 구현 내용을 자세하게 파악하고 싶다면? -> [노션 링크 바로가기](https://paper-mass-5ff.notion.site/8e55c81023334a869c8779094ba5f595?pvs=4) ### 각 페이지 별 헤더 사용...