[#412] [북카이브] 북카이브 페이지 (회원) #436

minjongbaek · 2023-11-09T15:17:57Z

구현 내용

북카이브 페이지 (회원) 을 디자인에 맞게 수정했어요.
북카이브 페이지에 있던 Chakra UI를 전부 걷어냈어요.
v1/bookShelf 에 위치한 BookShelf 컴포넌트 이름을 BookShelfCard 로 교체했어요.

스크린샷

pr 포인트

책을 추천하는 UI는 북카이브 페이지에서만 노출되고 있어 컴포넌트로 만들지 않았어요.

관련 이슈

Close [북카이브] 북카이브 페이지 (회원) #412

vercel · 2023-11-09T15:18:03Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
dadok	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 15, 2023 4:24pm

github-actions · 2023-11-09T15:20:36Z

🚀 Storybook: https://64aba10cd478ad51307a8997-jnojvbxtkl.chromatic.com/

minjongbaek · 2023-11-09T15:31:51Z

src/v1/book/BookCover.tsx

@@ -45,7 +45,7 @@ const BookCover = ({ src, title, size = 'medium' }: BookCoverProps) => {
  const sizeClasses = getCoverSizeClasses(size);

  return (
-    <span className={`relative ${sizeClasses}`}>
+    <div className={`relative ${sizeClasses}`}>


@gxxrxn 기존 코드에는 감싸는 요소가 <span /> 태그로 작성되어 있었는데 inline 요소를 사용하신 이유가 있을까요?

BookCover 컴포넌트가 display:inline 이기 때문에 사용하는 곳에서 크기를 명시적으로 정해줘야하는 문제가 있어서 <div /> 태그로 수정해두었어요. 👀

생각해보니 inline 요소에 크기 속성을 주고 있었군요..🫠 div 태그로 수정하면 좋을 것 같아요! 감사합니다! 👍🏻

hanyugeon

@minjongbaek

고생하셨습니다!
코멘트 확인해주세요~

+a) #412 #435 두 가지 이슈가 중복되고있습니다 확인해주세요!

hanyugeon · 2023-11-10T08:01:21Z

src/app/bookarchive/page.tsx

+    <div className="flex w-full flex-col gap-[1rem]">
+      <TopHeader pathname="/bookarchive"></TopHeader>
+      {/* TODO: 스켈레톤 컴포넌트로 교체 */}
+      <Suspense fallback={null}>
        <Contents />
      </Suspense>
-    </VStack>
+    </div>


p3;

현재 Component/ContextProvider.tsx에 <Layout />부분이 새로 만든 레이아웃으로 교체되지 않았습니다
만약 그부분이 교체된다면 <TopHeader /> 컴포넌트와 감싸고 있는 <div>요소는 따로 필요하지 않을 것 같아요!

@hanyugeon 혹시 이 부분이 오늘 스크럼 때 이야기 했던 내용과 연결되는 건가요? ContextProvider 에서 사용되는 Layout 컴포넌트가 교체되어야 한다는 말씀이신가요? 👀

@minjongbaek

ContextProvider 에서 사용되는 Layout 컴포넌트가 교체되어야 한다는 말씀이신가요? 👀

다시 검토해보니 아직 BottomNavigation에 BottomSheet를 띄우는 이벤트가 적용되지않아 부자연스러움이 있네요...!
나중에 새로운 이슈를 통해 일괄적으로 적용하는게 좋을 것 같아 보이네요.
이 부분은 나중에 수정해봅시다!!

@hanyugeon 확인했습니다. 다시 리뷰 부탁드려요~

@minjongbaek @hanyugeon

현재 Component/ContextProvider.tsx에 부분이 새로 만든 레이아웃으로 교체되지 않았습니다
만약 그부분이 교체된다면 <TopHeader /> 컴포넌트와 감싸고 있는 <div>요소는 따로 필요하지 않을 것 같아요!

재현님이 작업하신 Layout 컴포넌트에 TopHeader 컴포넌트가 포함되어있어서 해당 페이지에서는 제거하면 좋을 것 같다는 뜻으로 이해했어요!

그런데 문득.. layout에 TopHeader 컴포넌트가 포함되어있는게 어색한것 같기두 하네요..!! 모든 페이지에서 TopHeader가 노출되는 것이 아니기 때문에 매번 pathname을 비교해서 일부 페이지에서만 렌더링하는 것도 비효율적인 것 같다는 생각이 들어요. 혹시 이전에 이와 관련해서 이야기를 나눴던 적이 있을까요? 제가 놓친거라면 죄송해요..🥺

@gxxrxn @hanyugeon
아 재현님이 남긴 코멘트를 이제서야 제대로 이해한 것 같아요 😅
이전에 이야기 했을 때는 layout 에 TopHeader 컴포넌트를 포함하는 방향으로 작업하자고 했던 것 같아요. 이와 관련해서는 이번 주 회의 시간에 다시 이야기 해보는건 어떨까요? 👀

hanyugeon · 2023-11-11T08:22:53Z

src/v1/bookArchive/BookArchiveForAuth.tsx

+          <div key={bookId}>
+            <BookCover src={imageUrl} title={title} size="large" />
+            <div className="line-clamp-2 break-keep text-center text-xs font-normal">
+              {title}
+            </div>
+          </div>


p5;

key값만 부여받는 역할의 <div /> 요소라면 Fragments를 활용해 보아도 좋을 것 같습니다!

@hanyugeon 해당 div 요소는 key 역할 뿐 아니라 BookCover 와 title을 수직으로 배치하기 위한 역할도 수행하고 있어요. Fragments를 사용하게되면 트리의 계층 구조가 허물어져 레이아웃이 틀어질거에요.

p1;

책을 클릭하면 책 상세 페이지로 이동해야하지 않을까요?

아래 디자인처럼 이미지와 제목 사이에 1rem의 gap이 있으면 좋을 것 같아요!
.

개인적으로는 도서 제목의 행간이 넓어서 가독성이 좋지 않은 것 같아요. 제목 요소에 leading-tight 속성을 주는건 어떨까요?

드문 경우긴 하겠지만, 굉장히 긴 단어가 제목에 포함되면 레이아웃이 살짝 달라져요. 간단하게는 wrapper 요소에 max-widht: 9rem을 적용하면 해결할 수 있을 것 같아요!

@gxxrxn 가장 중요한 동작을 빼먹었네요 😅
355857b 반영해두었습니다.

제가 디테일하게 보지못한 부분까지 규란님이 잘 캐치해주셨군요..! 💯 👍 🙇

hanyugeon · 2023-11-11T08:53:54Z

src/v1/bookShelf/BookShelfCard.tsx

+const BookShelf = ({
+  bookshelfId,
+  bookshelfName,
+  books,
+  likeCount,
+}: APIBookshelf) => {


comment;

현재 BookShelfCard 컴포넌트에 BookShelf, Book 컴포넌트가 한꺼번에 정의되어 사용되고 있습니다!

이후 북카이브 뿐만이 아닌 사용자 책장 페이지, 프로필 페이지에서도 사용하기 용이한 구조를 만들 수 있도록 고민해보아도 좋을 것 같아요!

마크업 및 API연동은 잘 구현해 주신것같아요! 고생하셨어요!!

@hanyugeon 재현님 의견에 동의해요. 북카이브와 책장 페이지에서 보여지는 모습이 미세하게 달라서 어떻게 추상화를 할 지 고민하다가 우선 작업 목적에 맞게끔 구현해두었어요.
다음 책장 페이지를 작업하게 된다면 의견 반영하도록 하겠습니다~

gxxrxn

저도 시맨틱 태그를 잘 사용하는 편은 아니지만.. 전체적으로 div 태그만을 사용해서 마크업하신 특별한 이유가 있을까요? 개인적으로는 특히나 tailwind를 사용하고 있기 때문에 적절한 컴포넌트 분리와 시맨틱한 태그를 사용하는 것이 코드 가독성을 높일 수 있는 방법 중 하나라고 생각하고 있어요! SEO를 고려해봤을 때도 👀 이런 책들이 많이 꽂혔어요, 🔥 인기 책장과 같은 문구에는 heading 태그를 적용해도 좋지 않을까요??

수고많으셨습니다! 👍🏻

gxxrxn · 2023-11-15T12:51:37Z

src/app/bookarchive/page.tsx

+    <div className="flex w-full flex-col gap-[1rem]">
+      <TopHeader pathname="/bookarchive"></TopHeader>
+      {/* TODO: 스켈레톤 컴포넌트로 교체 */}
+      <Suspense fallback={null}>
        <Contents />
      </Suspense>
-    </VStack>
+    </div>


@minjongbaek @hanyugeon

현재 Component/ContextProvider.tsx에 부분이 새로 만든 레이아웃으로 교체되지 않았습니다
만약 그부분이 교체된다면 <TopHeader /> 컴포넌트와 감싸고 있는 <div>요소는 따로 필요하지 않을 것 같아요!

재현님이 작업하신 Layout 컴포넌트에 TopHeader 컴포넌트가 포함되어있어서 해당 페이지에서는 제거하면 좋을 것 같다는 뜻으로 이해했어요!

그런데 문득.. layout에 TopHeader 컴포넌트가 포함되어있는게 어색한것 같기두 하네요..!! 모든 페이지에서 TopHeader가 노출되는 것이 아니기 때문에 매번 pathname을 비교해서 일부 페이지에서만 렌더링하는 것도 비효율적인 것 같다는 생각이 들어요. 혹시 이전에 이와 관련해서 이야기를 나눴던 적이 있을까요? 제가 놓친거라면 죄송해요..🥺

gxxrxn · 2023-11-15T15:16:09Z

src/v1/bookArchive/BookArchiveForAuth.tsx

+          <div key={bookId}>
+            <BookCover src={imageUrl} title={title} size="large" />
+            <div className="line-clamp-2 break-keep text-center text-xs font-normal">
+              {title}
+            </div>
+          </div>


p1;

책을 클릭하면 책 상세 페이지로 이동해야하지 않을까요?

아래 디자인처럼 이미지와 제목 사이에 1rem의 gap이 있으면 좋을 것 같아요!
.

개인적으로는 도서 제목의 행간이 넓어서 가독성이 좋지 않은 것 같아요. 제목 요소에 leading-tight 속성을 주는건 어떨까요?

드문 경우긴 하겠지만, 굉장히 긴 단어가 제목에 포함되면 레이아웃이 살짝 달라져요. 간단하게는 wrapper 요소에 max-widht: 9rem을 적용하면 해결할 수 있을 것 같아요!

minjongbaek · 2023-11-15T16:27:55Z

전체적으로 div 태그만을 사용해서 마크업하신 특별한 이유가 있을까요? 개인적으로는 특히나 tailwind를 사용하고 있기 때문에 적절한 컴포넌트 분리와 시맨틱한 태그를 사용하는 것이 코드 가독성을 높일 수 있는 방법 중 하나라고 생각하고 있어요! SEO를 고려해봤을 때도 👀 이런 책들이 많이 꽂혔어요, 🔥 인기 책장과 같은 문구에는 heading 태그를 적용해도 좋지 않을까요??

@gxxrxn 특별한 이유는 없고, 제가 시맨틱한 마크업에 대해 잘 알지 못해서 div 태그를 주로 사용했어요. 355857b dea1521 작업을 진행하면서 의견 주신대로 heading 태그로 교체하고 책을 표현하는 요소는 list 태그를 적용해봤어요.

꼼꼼한 리뷰 덕분에 좋은 코드에 대한 고민을 다시 할 수 있었어요. 다시 리뷰 부탁드리며, 시맨틱하지 못한 마크업이 있다면 말씀해주세요!

gxxrxn

적극적으로 피드백 반영해주셔서 감사해요!! 수고 많으셨습니다~!! 👍🏻

gxxrxn · 2023-11-16T12:48:22Z

src/ui/Base/TopHeader.tsx

+      <h1 className="text-xl font-bold text-main-900">
        {getHeaderLabel(pathname)}
-      </p>
+      </h1>


너무 좋은데요?? 👍🏻

hanyugeon · 2023-11-16T15:48:20Z

@gxxrxn

그런데 문득.. layout에 TopHeader 컴포넌트가 포함되어있는게 어색한것 같기두 하네요..!! 모든 페이지에서 TopHeader가 노출되는 것이 아니기 때문에 매번 pathname을 비교해서 일부 페이지에서만 렌더링하는 것도 비효율적인 것 같다는 생각이 들어요. 혹시 이전에 이와 관련해서 이야기를 나눴던 적이 있을까요? 제가 놓친거라면 죄송해요..🥺

이전에 이야기를 나누었던 적은 없던걸로 기억해요..!

Layout 컴포넌트를 작성할 때 원래 BottomNavigation만 포함시키는것이였는데
pathname 비교를 통해 보여주는 조건이 TopHeader도 BottomNavigation과 동일하여
TopHeader를 추가해서 작업하게 되었습니다.. 🥺

hanyugeon

고생하셨습니다!! 👍

minjongbaek · 2023-11-16T16:01:38Z

@hanyugeon 그렇다면, 일단 해당 PR 이대로 머지하고 주말에 Layout 관련해서 이야기를 나눠보면 될까요~?

hanyugeon · 2023-11-16T16:07:21Z

@minjongbaek

저는 일단 approve를 받아놓은 상태에서 회의 이후 머지하는게 좋을 것 같아요!
(첫 페이지 작업인 만큼 신중하게 되네요 ㅎ...)
다른 리뷰어분 생각은 어떠신가요 @gxxrxn 👀

gxxrxn · 2023-11-16T16:22:53Z

@minjongbaek @hanyugeon

논의 후에 Layout 컴포넌트가 수정될 여지가 있을 것 같아서, 이 pr은 머지하고 새로운 이슈 생성해서 한번에 작업해도 좋을 것 같아요!

This reverts commit 434dd73.

* Revert "[#634] v1 디렉토리 네이밍 수정 (#635)" This reverts commit 86c304a. * Revert "[#632] 사용하지 않는 라이브러리 제거 (#633)" This reverts commit 34b4efb. * Revert "[#627] 불필요한 utils, hooks, constants 정리 및 import문 통일 (#631)" This reverts commit 254f6bb. * Revert "[#629] tailwind config color 정리 (#630)" This reverts commit 595f476. * Revert "[#622] BottomActionButton 패딩 수정 (#626)" This reverts commit 9796dab. * Revert "[#625] Toast 컴포넌트 icon이 잘리는 문제 해결 (#628)" This reverts commit e868efc. * Revert "[#623] [도서 검색] Input 검색 아이콘 Layout Shift 해결 (#624)" This reverts commit d0392d6. * Revert "[#619] PWA 구현 (#621)" This reverts commit 587f990. * Revert "[#616] 책장 좋아요 뱃지 UI 수정 (#620)" This reverts commit 70af27a. * Revert "[#349] 메타 데이터 추가 (#617)" This reverts commit 209ba3e. * Revert "[#556] 버튼 컴포넌트 disabled style 작성 (#615)" This reverts commit 0425223. * Revert "[#610] 로그인 페이지 구현 (#614)" This reverts commit badcf8e. * Revert "[#608] 페이지 접근 권한 부여 (#611)" This reverts commit 7f1cd21. * Revert "[#612] 북카이브 페이지에서 발생하는 무한 리랜더링 버그 수정 (#613)" This reverts commit bac4ff2. * Revert "[#605] Google LightHouse CI 추가 (#606)" This reverts commit d6cccdb. * Revert "[#602] [모임 생성] 퍼널 progress bar, stepper 구현 (#607)" This reverts commit bcbdc67. * Revert "[#603] [내 프로필] ErrorBoundary 제거 (#604)" This reverts commit f044cd2. * Revert "feat: start-ssl script 작성 (#601)" This reverts commit 8de7079. * Revert "[#596] accessToken이 만료된 직후 다른 페이지 접근할 때 에러 페이지를 보여주지 않고 새로고침 (#600)" This reverts commit 59407c6. * Revert "[#592] [도서 검색] 검색 키워드 및 결과 유지 기능 구현 (#593)" This reverts commit af2413c. * Revert "[#595] Toast UI 개선 (#597)" This reverts commit 52e9992. * Revert "[#591] 3D 책 컴포넌트, 책장 페이지 개선 (#594)" This reverts commit 2beb0f2. * Revert "[#570] [도서 검색] 도서 검색 결과 렌더링 시 search 헤더 숨기고 input sticky 적용 (#589)" This reverts commit 0ff4ca0. * Revert "[#588] 폰트 디자인 시스템 적용 (#590)" This reverts commit 6a054a4. * Revert "[#557] [도서 검색] 최근 검색어 api 요청 (#587)" This reverts commit 591e9a8. * Revert "[#585] [프로필 페이지] '참여한 모임' 영역 사용자가 모임장인 모임에 마크 추가 (#586)" This reverts commit 3a3f263. * Revert "[#575] Drawer 컴포넌트 개선 (#576)" This reverts commit cff01fd. * Revert "[#581] [모임 생성] 모임 생성 퍼널 페이지 작성 (#582)" This reverts commit 55a08ee. * Revert "[#583] Switch 컴포넌트 style 수정 (#584)" This reverts commit e7798f7. * Revert "[#578] iOS 환경에서의 DatePicker UI 수정 (#579)" This reverts commit 24b42bf. * Revert "[#574] FloatingButton 버그 수정 (#577)" This reverts commit dd37ff5. * Revert "[#559] BookInfoCard 컴포넌트 수정 및 goToBookSelectStep 이벤트 추가 (#572)" This reverts commit f1c6a3c. * Revert "[#571] Drawer 컴포넌트 style 개선 (#573)" This reverts commit f21ada4. * Revert "[#561] [모임 생성] 모임 상세 퍼널 개선 (#567)" This reverts commit bbd1123. * Revert "[#552] next custom server 구동 시 hmr이 동작하지 않는 문제 해결 (#569)" This reverts commit 019abd5. * Revert "[#563] [모임 생성] 모임 가입문제 퍼널 (#568)" This reverts commit 4d6a14f. * Revert "[#564] BottomActionButton 컴포넌트 position 속성 수정 (#566)" This reverts commit 94365ee. * Revert "[#562] [모임 생성] 모임 이름 퍼널 (#565)" This reverts commit 0bd1349. * Revert "[#543] [모임 생성] 책 선택 퍼널 (#560)" This reverts commit b2f5236. * Revert "[#553] [모임 생성] 모임 상세 퍼널 (#558)" This reverts commit 31a6d84. * Revert "[#554] 스토리북 preview decorator에서 Layout 제거 (#555)" This reverts commit 495321e. * Revert "[#549] 'unable to verify first certificate' 에러 해결 (#550)" This reverts commit bdfa458. * Revert "fix: 모바일 환경에서의 스크롤 버그 수정 (#551)" This reverts commit 1d56794. * Revert "[#546] [독서모임] 모임 페이지 누락된 작업 및 버그 수정 (#547)" This reverts commit 448e8b5. * Revert "[#531] 로컬 서버에 https 적용 (#541)" This reverts commit f2f4a82. * Revert "[#540] [모임 생성] 모임 상세 퍼널 마크업 (#542)" This reverts commit 5d23ab3. * Revert "[#544] vscode react component snippet 작성 (#545)" This reverts commit 68dc181. * Revert "[#535] [독서모임] 독서모임 목록 페이지 개선 (#539)" This reverts commit 56137df. * Revert "[#505] [모임 상세] 모임 삭제 기능 구현 (#537)" This reverts commit efdb2c5. * Revert "[#530] FloatingButton 컴포넌트 적용 (#536)" This reverts commit 3671624. * Revert "[#529] redirect 페이지 개선, 토큰 업데이트 로직 개선 (#532)" This reverts commit 0867ee4. * Revert "[#523] [모임 상세] 모임 수정 페이지 (#527)" This reverts commit f889a79. * Revert "[#526] 도서 검색 페이지 개선 (#528)" This reverts commit 81b39b7. * Revert "[#524] Menu 컴포넌트 개선 (#525)" This reverts commit 52687da. * Revert "[#521] TextArea 컴포넌트 (#522)" This reverts commit a1673f6. * Revert "[#518] Input 컴포넌트 style 추가 (#519)" This reverts commit 2bd859a. * Revert "[#516] DatePicker 컴포넌트 (#520)" This reverts commit 63af14d. * Revert "[#504] [모임 상세] 모임 게시글 작성 기능 구현 (#517)" This reverts commit e6f8d87. * Revert "[#509] [레이아웃] BottomNavigation 버그 및 UI 수정 (#510)" This reverts commit ada909a. * Revert "feat: 모임 게시글 수정, 삭제 기능 구현 (#513)" This reverts commit 36c1e71. * Revert "[#507] [프로필] 로그아웃 햄버거 구현 (#508)" This reverts commit 90c5855. * Revert "feat: hosts 파일을 수정할 수 있는 updateDevHost script 작성 (#511)" This reverts commit cd297e9. * Revert "[#497] [책 상세] 책 상세 페이지 api 연결 (#500)" This reverts commit 7842028. * Revert "[#498] useFunnel 작성 (#501)" This reverts commit e6f7878. * Revert "[#489] [도서 검색] 도서검색 페이지 개선 작업 (#490)" This reverts commit 0043fe7. * Revert "[#491] [도서 검색] 베스트셀러 클릭 시 라우팅 경로 수정 (알라딘 -> 다독다독) (#492)" This reverts commit e68d1d3. * Revert "[#493] 에러 페이지 (#494)" This reverts commit e95c940. * Revert "[#495] [책 상세] 책 상세 페이지 마크업 (#496)" This reverts commit 052e6d4. * Revert "[#474] 리프레시 토큰이 만료된 경우 에러 페이지로 넘어가는 이슈 수정 (#483)" This reverts commit 633da9f. * Revert "[#486] [책 상세 / 모임 상세] 코멘트 목록 컴포넌트 (#487)" This reverts commit f452642. * Revert "[#443] TopNavigation 컴포넌트 스타일 fixed 적용 (#488)" This reverts commit 60d3311. * Revert "[#468] [검색] 검색 페이지 작성 (#478)" This reverts commit 313d61a. * Revert "[#481] [책 상세] 책 정보 컴포넌트 (#482)" This reverts commit 03fa16e. * Revert "[#442] 스켈레톤 컴포넌트 작성 (#480)" This reverts commit 22b06a2. * Revert "[#453] [모임] 모임 목록 페이지 개선 (#475)" This reverts commit c6b1ab1. * Revert "typo: 오타 수정 (#485)" This reverts commit 395e050. * Revert "[#476] [모임 상세] 비로그인 시 모임 참여하기 버튼 비활성화 (#479)" This reverts commit a2e628d. * Revert "[#472] Drawer 컴포넌트 (#477)" This reverts commit 5e54b72. * Revert "[#470] Menu 컴포넌트 (#471)" This reverts commit 0a4705d. * Revert "[#465] [모임] BookInfoCard 컴포넌트 추상화 (#466)" This reverts commit 81bc6d8. * Revert "[#467] 내가 가입한 모임 상세 페이지 (#469)" This reverts commit 75070ce. * Revert "[#463] Loading 컴포넌트 animation 버그 해결 (#464)" This reverts commit 8219f67. * Revert "[#456] [프로필] 프로필 페이지 (#461)" This reverts commit 150592c. * Revert "[#455] [모임 상세] 모임 가입 문제 페이지 (#462)" This reverts commit 7bbdbaf. * Revert "[#459] [유저 로그인] 로그인 BottomSheet 컴포넌트 작성 (#460)" This reverts commit da3fb33. * Revert "[#457] LikeButton 컴포넌트 (#458)" This reverts commit 9d1a5c7. * Revert "베이스 컴포넌트 위치 변경 (#454)" This reverts commit fd8ad31. * Revert "[#447] [모임 상세] 모임 참여 여부, 모임장 여부에 따른 UI 구현 (#450)" This reverts commit 61ada97. * Revert "[#449] [책장] 책장 상세 페이지 (#451)" This reverts commit 1f389e4. * Revert "[#445] Layout 컴포넌트에서 TopHeader 컴포넌트 분리 (#452)" This reverts commit 9d11c70. * Revert "[#415] [프로필] 프로필 생성 페이지 (#427)" This reverts commit 4175612. * Revert "[fix] group 페이지 타입 에러 수정 (#448)" This reverts commit 66005ea. * Revert "[#434] [모임 상세] 독서모임 상세 페이지 api 연결 (#441)" This reverts commit 617614f. * Revert "[#439] [독서모임] 모임 목록 페이지 리팩토링 (#440)" This reverts commit f50f393. * Revert "[#414] [프로필] 프로필 수정 페이지 (#426)" This reverts commit 91b8171. * Revert "[#437] [독서모임] Avartar, bookGroupStatus 적용 및 typo 수정 (#438)" This reverts commit b56d3d5. * Revert "[#412] [북카이브] 북카이브 페이지 (회원) (#436)" This reverts commit 434dd73. * Revert "[refactor] bookcover v1 폴더로 이동 (#433)" This reverts commit 434dd8c. * Revert "[refactor] 폴더 및 파일 구조, 네이밍 수정 (#432)" This reverts commit 5ba2e21. * Revert "[fix] github 에서 대소문자 구문하지 못하는 문제 해결 (#431)" This reverts commit 549a5e9. * Revert "[#403] [모임 상세] 모임 정보 컴포넌트 (#423)" This reverts commit 378828c. * Revert "fix: 이벤트 핸들러 명칭 변경" This reverts commit de6588c. * Revert "fix: 컴포넌트명 변경" This reverts commit 03899d7. * Revert "fix: 스타일 관련 코드 수정 및 불필요한 요소 제거" This reverts commit 14a6b84. * Revert "feat: 내가 가입한 모임 simple ui 구현" This reverts commit 5dd3e60. * Revert "[#410] [북카이브] 북카이브 페이지 (비회원) (#425)" This reverts commit 08c919d. * Revert "[#411][독서 모임] Detail 독서 모임 컴포넌트 (#428)" This reverts commit cbb032a. * Revert "[#408] [모임 상세] 게시글 컴포넌트 (#429)" This reverts commit 2277b0d. * Revert "[#404][독서모임] 모임 검색 컴포넌트 (#419)" This reverts commit 281beee. * Revert "[#418] Avatar 컴포넌트 (#420)" This reverts commit 51d471e. * Revert "[#406] [모임 상세] 멤버 목록 컴포넌트 (#424)" This reverts commit 199a234. * Revert "[#399] Toast 컴포넌트 (#402)" This reverts commit 9333f87. * Revert "[#405] [레이아웃] 공통 레이아웃 (#421)" This reverts commit fb8d8d6. * Revert "[#398] RadioButton 컴포넌트 (#400)" This reverts commit 9c8bdc2. * Revert "[#413] Issue 템플릿에 v1.0 라벨 추가 (#417)" This reverts commit facb746. * Revert "[#363] BottomActionButton 컴포넌트 (#395)" This reverts commit a653d3b. * Revert "[#366] BottomSheet 컴포넌트 (#394)" This reverts commit a43dea4. * Revert "[#364] Badge 컴포넌트 (#396)" This reverts commit 86edc76. * Revert "[#362] TopNavigation 컴포넌트 리팩터링 (#393)" This reverts commit 55a6b98. * Revert "[#361] BottomNavigation 컴포넌트 (#392)" This reverts commit d7d8ca6. * Revert "[#388] Input, Select 컴포넌트 에러 처리 (#389)" This reverts commit f5bf5ef. * Revert "[#390] storybook snippet 작성 (#391)" This reverts commit 97984a7. * Revert "[#365] Modal 컴포넌트 (#387)" This reverts commit 8e931fc. * Revert "[#367] Switch 컴포넌트 (#386)" This reverts commit e1b1ae9. * Revert "[#382] Assets 정리 (#384)" This reverts commit 182def7. * Revert "[#383] Select 컴포넌트 (#385)" This reverts commit 7c189bc. * Revert "[#357] TopHeader UI 컴포넌트 및 스토리북 작성 (#379)" This reverts commit 4dfbcc8. * Revert "[#358] TopNavigation 컴포넌트 (#378)" This reverts commit 41f8f99. * Revert "[#360] Input 컴포넌트 작성 (#380)" This reverts commit e28792d. * Revert "[#359] Button 컴포넌트 (#381)" This reverts commit 6b84755. * Revert "[#376] Storybook 설정 및 GlobalTheme, Font 수정 (#377)" This reverts commit 3a7a9e6. * Revert "[#374] Tailwind 클래스를 정렬하기 위한 Prettier 플러그인 추가 (#375)" This reverts commit e365160. * Revert "[#372] Theme, GlobalStyle, Font 세팅 (#373)" This reverts commit 4f7df42. * Revert "[#370] Storybook & Headless UI 추가 (#371)" This reverts commit 7ad8aef.

minjongbaek added 4 commits November 9, 2023 22:54

style: BookCover 컴포넌트에서 span 대신 div 태그를 쓰도록 수정

53caca1

style: 북카이브 페이지 레이아웃 변경

3395338

refactor: 북카이브 페이지에서 ChakraUI 제거

08f3460

refactor: BookShelf 에서 BookShelfCard 로 컴포넌트 이름 변경

8fe6865

minjongbaek linked an issue Nov 9, 2023 that may be closed by this pull request

[북카이브] 북카이브 페이지 (회원) #435

Closed

minjongbaek commented Nov 9, 2023

View reviewed changes

minjongbaek requested review from gxxrxn, WooDaeHyun and hanyugeon November 9, 2023 15:32

minjongbaek added 🐥 프론트 필수! ✨ feature New feature or request 🔥 v.1.0 New feature for releasing v.1.0 labels Nov 9, 2023

minjongbaek self-assigned this Nov 9, 2023

hanyugeon reviewed Nov 11, 2023

View reviewed changes

minjongbaek changed the title ~~[#435] [북카이브] 북카이브 페이지 (회원)~~ [#412] [북카이브] 북카이브 페이지 (회원) Nov 12, 2023

minjongbaek removed a link to an issue Nov 12, 2023

[북카이브] 북카이브 페이지 (회원) #435

Closed

minjongbaek mentioned this pull request Nov 12, 2023

[#434] [모임 상세] 독서모임 상세 페이지 api 연결 #441

Merged

gxxrxn requested changes Nov 15, 2023

View reviewed changes

refactor: div 태그를 시맨틱한 태그로 교체

355857b

vercel bot deployed to Preview November 15, 2023 16:14 View deployment

refactor: TopHeader 컴포넌트에서 h1 태그를 사용하도록 수정

dea1521

vercel bot deployed to Preview November 15, 2023 16:24 View deployment

minjongbaek requested review from hanyugeon and gxxrxn November 15, 2023 16:28

gxxrxn approved these changes Nov 16, 2023

View reviewed changes

hanyugeon approved these changes Nov 16, 2023

View reviewed changes

minjongbaek merged commit 699b1aa into main Nov 16, 2023
3 checks passed

minjongbaek deleted the feat/#435 branch November 16, 2023 16:52

gxxrxn pushed a commit that referenced this pull request Jun 17, 2024

[#412] [북카이브] 북카이브 페이지 (회원) (#436)

434dd73

gxxrxn added a commit that referenced this pull request Aug 20, 2024

Revert "[#412] [북카이브] 북카이브 페이지 (회원) (#436)"

9709761

This reverts commit 434dd73.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[#412] [북카이브] 북카이브 페이지 (회원) #436

[#412] [북카이브] 북카이브 페이지 (회원) #436

minjongbaek commented Nov 9, 2023 •

edited

Loading

vercel bot commented Nov 9, 2023 •

edited

Loading

github-actions bot commented Nov 9, 2023 •

edited

Loading

minjongbaek Nov 9, 2023

gxxrxn Nov 15, 2023

hanyugeon left a comment •

edited

Loading

hanyugeon Nov 10, 2023

minjongbaek Nov 12, 2023 •

edited

Loading

hanyugeon Nov 14, 2023

minjongbaek Nov 15, 2023

gxxrxn Nov 15, 2023

minjongbaek Nov 15, 2023

hanyugeon Nov 11, 2023

minjongbaek Nov 12, 2023

gxxrxn Nov 15, 2023

minjongbaek Nov 15, 2023

hanyugeon Nov 16, 2023

hanyugeon Nov 11, 2023

minjongbaek Nov 12, 2023

gxxrxn left a comment •

edited

Loading

gxxrxn Nov 15, 2023

gxxrxn Nov 15, 2023

minjongbaek commented Nov 15, 2023 •

edited

Loading

gxxrxn left a comment

gxxrxn Nov 16, 2023

hanyugeon commented Nov 16, 2023

hanyugeon left a comment

minjongbaek commented Nov 16, 2023

hanyugeon commented Nov 16, 2023

gxxrxn commented Nov 16, 2023

[#412] [북카이브] 북카이브 페이지 (회원) #436

[#412] [북카이브] 북카이브 페이지 (회원) #436

Conversation

minjongbaek commented Nov 9, 2023 • edited Loading

구현 내용

스크린샷

pr 포인트

관련 이슈

vercel bot commented Nov 9, 2023 • edited Loading

github-actions bot commented Nov 9, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hanyugeon left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

minjongbaek Nov 12, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

gxxrxn left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

minjongbaek commented Nov 15, 2023 • edited Loading

gxxrxn left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hanyugeon commented Nov 16, 2023

hanyugeon left a comment

Choose a reason for hiding this comment

minjongbaek commented Nov 16, 2023

hanyugeon commented Nov 16, 2023

gxxrxn commented Nov 16, 2023

minjongbaek commented Nov 9, 2023 •

edited

Loading

vercel bot commented Nov 9, 2023 •

edited

Loading

github-actions bot commented Nov 9, 2023 •

edited

Loading

hanyugeon left a comment •

edited

Loading

minjongbaek Nov 12, 2023 •

edited

Loading

gxxrxn left a comment •

edited

Loading

minjongbaek commented Nov 15, 2023 •

edited

Loading