triple-frontend
triple-frontend copied to clipboard
Monorepo for Triple Frontend Components and Libraries
tna에서 선행작업 진행했던 [요 작업](https://github.com/titicacadev/tna-web-v2/pull/508)을 TF로 이전합니다. TF로 이전할려고 보니 좀 고민스러운 포인트들이 있군요..
## 배경 > 초기 버젼의 생각입니다. 종종 필터 팝업다이얼로그나 안내에 bottom-up 방식의 팝업 다이얼로그를 띄우는 기능들을 구현하고 있는데요. 보통 다음과 같은 절차가 요구됩니다. 1. 팝업에 오픈 상태를 위해 hash 상태값을...
관련 스레드: https://titicaca.slack.com/archives/CEEPB4TDY/p1606891884283000?thread_ts=1606890617.281500&cid=CEEPB4TDY > 이거 보다가 발견한건데요 ㅋㅋ > 공유웹 장소상세>리뷰>사진선택하면 리뷰팝업하고 사진팝업 2개가 뜨네영(리뷰팝업 닫으면 뒤에 사진팝업 남아있음) 원인을 살펴보니 Review entry 컨테이너에 포함된 Image 요소의 click 이벤트가 Image...
항공 웹과 호텔 웹에서 사용하고 있는 컴포넌트입니다. 사용 방식이 미묘하게 달라 공통 로직을 분리하기 힘들겠지만, 컴포넌트로 만들면 선 위에 점을 찍는 복잡한 절차를 거치지 않아 앞으로 구현에 큰 도움이 될...
## 문제점 [CHANGELOG를 각 PR에서 작성하는 방법이 불편](https://github.com/titicacadev/triple-frontend/pull/1221#issuecomment-784965908)합니다. 모든 Pull Request에서 CHANGELOG.md 파일이 바뀌기 때문에 하나의 PR이 머지되면 다른 PR은 무조건 충돌이 생깁니다. 또한, 매 릴리즈마다 CHANGELOG에 빼먹은 것은 없는지...
ListTopBanners, ContentDetailsBanner에서 사용되는 API의 endpoint, requestBody가 다름에도 하나의 function에 묶여있습니다. AdBanner Service 타입이 점점 많아지게되면 코드가 복잡해질 우려가있습니다. 타입별로 Service를 분리가 필요해보입니다. - [관련 코멘트](https://github.com/titicacadev/triple-frontend/pull/1099#pullrequestreview-540648905)
slider의 lowerbound, upperbound knob이 가까운 값으로 겹치면 z-index가 더 높은 knob만 터치가 가능해집니다. 일부 상황에서 사용성에 문제가 있는 동작이어서 개선이 필요해 보입니다. 관련 이슈: https://titicaca.atlassian.net/browse/TAQ-313?atlOrigin=eyJpIjoiNDFlYjZmNWMwNGNjNGZiMmEwMjIwZGJjMzMzNDc4ODciLCJwIjoiamlyYS1zbGFjay1pbnQifQ&focusedCommentId=20972&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-20972 - 아이디어 1: 최근에 값을...
This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [@sentry/nextjs](https://togithub.com/getsentry/sentry-javascript/tree/master/packages/nextjs) ([source](https://togithub.com/getsentry/sentry-javascript)) | devDependencies | major | [`7.118.0` -> `8.26.0`](https://renovatebot.com/diffs/npm/@sentry%2fnextjs/7.118.0/8.26.0) | --- ###...
This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [@sentry/nextjs](https://togithub.com/getsentry/sentry-javascript/tree/master/packages/nextjs) ([source](https://togithub.com/getsentry/sentry-javascript)) | devDependencies | minor | [`7.118.0` -> `7.119.0`](https://renovatebot.com/diffs/npm/@sentry%2fnextjs/7.118.0/7.119.0) | --- ###...
This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [eslint](https://eslint.org) ([source](https://togithub.com/eslint/eslint)) | devDependencies | major | [`^8.57.0` -> `^9.9.0`](https://renovatebot.com/diffs/npm/eslint/8.57.0/9.9.0) | --- ###...