[ feat ] GTM 페이지별 트래킹 #232
Conversation
| <St.CategoryLink className={GTM_CLASS_NAME.cardKeep} to={routePaths.Category}> | ||
| <IcNextCardBtn /> | ||
| </St.CategoryLink> |
There was a problem hiding this comment.
클래스네임을 부모에만 적용하면, Icon을 누를 때 트랙킹되지 않을 거에요!
그 하위 요소까지, svg 파일 내의 path 태그에까지 클래스네임을 부여해줘야 트랙킹되는 걸로 알고 있어요
There was a problem hiding this comment.
이에 대해 승헌이와 나눴던 슬랙 스레드가 있었는데, 궁금하면 찾아보셔도 좋을 것 같습니다!
There was a problem hiding this comment.
common에 있는 modal 컴포넌트도 마찬가지로 svg 파일에 className을 주려고 했는데, 그 아이콘은 공용으로 사용하고 있어서요! 이럴때에는 어떤 방법이 있을까요?-?
There was a problem hiding this comment.
😦😦😦😦 gtm용 svg를 따로 복제하는 건 어떨까요??_?
There was a problem hiding this comment.
모달의 x icon을 말하는건데용~! 카드 필터 모달에서도 className으로 사용중이더라구요
복제를 각 모달 별로 해볼까요?!
There was a problem hiding this comment.
당장은 그래도 괜찮을 것 같아요 !! 후에 더 방법 생각나면 리팩토링 할 수 있으면 하고!
| <St.MedleyWrapper | ||
| bgcolorId={medleyCard._id} | ||
| className={GTM_CLASS_NAME[`medley${medleyCard.sticker.slice(0, 2)}`]} | ||
| onClick={toggleModal}> | ||
| <St.ContentTag>{medleyCard.sticker}</St.ContentTag> | ||
| <St.ContentTitle>{medleyCard.coverTitle}</St.ContentTitle> | ||
| </St.MedleyWrapper> |
There was a problem hiding this comment.
이것도 !
| <St.MedleyWrapper | |
| bgcolorId={medleyCard._id} | |
| className={GTM_CLASS_NAME[`medley${medleyCard.sticker.slice(0, 2)}`]} | |
| onClick={toggleModal}> | |
| <St.ContentTag>{medleyCard.sticker}</St.ContentTag> | |
| <St.ContentTitle>{medleyCard.coverTitle}</St.ContentTitle> | |
| </St.MedleyWrapper> | |
| <St.MedleyWrapper | |
| bgcolorId={medleyCard._id} | |
| className={GTM_CLASS_NAME[`medley${medleyCard.sticker.slice(0, 2)}`]} | |
| onClick={toggleModal}> | |
| <St.ContentTag | |
| className={GTM_CLASS_NAME[`medley${medleyCard.sticker.slice(0, 2)}`]} | |
| >{medleyCard.sticker}</St.ContentTag> | |
| <St.ContentTitle | |
| className={GTM_CLASS_NAME[`medley${medleyCard.sticker.slice(0, 2)}`]} | |
| >{medleyCard.coverTitle}</St.ContentTitle> | |
| </St.MedleyWrapper> | |
There was a problem hiding this comment.
medleyCard.sticker.slice(0, 2)} / 혹은 medley${medleyCard.sticker.slice(0, 2)} 가 중복되는데, 이를 컴포넌트 상단에 상수화해도 괜찮을 것 같아요
There was a problem hiding this comment.
옷!! 상수화하는게 더 보기 깔끔할 것 같네여ㅎㅎ 반영하겠습니다아~!
| <St.GrayRoot> | ||
| <St.GrayRoot className={closeOpacityClassName}> |
| closeOpacityClassName={GTM_CLASS_NAME[`medleyPreviewClose${medleyCard.sticker.slice(0, 2)}`]} | ||
| closeBtnClassName={GTM_CLASS_NAME[`medleyPreviewClose${medleyCard.sticker.slice(0, 2)}`]} |
There was a problem hiding this comment.
medleyCard.sticker.slice(0, 2)} 혹은 `medleyPreview${medleyCard.sticker.slice(0, 2)} 가 중복되는데
컴포넌트 상단에 상수로 표현함이 어떨까 합니다
|
아르바이트 중이라 10시 전에 반영해놓겠습니당 :D |
|
상수화 한 부분 반엉 감사합니다!! 근데 문자열에 템플릿 리터럴을 안 쓴 이유가 있을까요????! |
|
@joohaem 프리뷰 모달은 className이 |
📌 내용