Replies: 8 comments 28 replies
-
노아 - Tab Component |
Beta Was this translation helpful? Give feedback.
-
긍정적인 리뷰
개선 제안
|
Beta Was this translation helpful? Give feedback.
-
푸만능긍정적인 리뷰
체크한 리뷰 항목
개선 제안
<Tabs scrollButtons={false} direction={"horizontal"} centered={true}>
<Tabs.Panel
style={{ backgroundColor: "red" }}
label="별 하나의"
>
</Tabs>
|
Beta Was this translation helpful? Give feedback.
-
긍정적인 리뷰여러 환경에서 잘 보이는가?크롬, 파이어폭스, 사파리에서 확인을 해보았습니다. 여러 환경에서 시험을 해보았을 때, 잘 나오는 것을 확인할 수 있었습니다. 👍🏻👍🏻👍🏻 문서가 가독성 있게 쓰여있는가?문서의 경우 사용자가 바로 적용할 수 있도록 잘 쓰여져 있었습니다. 다만 사용 예시에는 완전 기본적인 예시만 나와있는데, 각각의 Props에 맞는 예시도 함께 작성해주시면 더욱 빠르게 해당 라이브러리를 사용할 수 있을 거 같습니다. ‘유연’하게 재사용 하기 좋은가?탭의 수가 많은 경우 스크롤을 통해 나머지 탭도 확인 가능하게 해주신 부분이 고민을 많이 하신 흔적이 느껴져서 매우 좋았습니다. 😆 다만 아이콘과 문자열이 함께 들어가면 아이콘이 들어간 부분이 개행이 되네용...?! 이 부분은 어쩔 수 없는 거 같지만... 생각을 해보시면 좋을 거 같습니당 😊 개선 제안매우 완벽하지만...! accentColor props의 default가 #3b82f6로 되어있더라구용! 물론 Hex 값을 이용해서 수정할 수 있는 부분은 매우 좋지만, 제 생각에는 색상을 미리 정의하여 사용자에게 선택지를 주는 방법도 좋을 거 같다는 생각을 하였습니당 😁 |
Beta Was this translation helpful? Give feedback.
-
긍정적인 리뷰
개선 제안
|
Beta Was this translation helpful? Give feedback.
-
긍정적인 코멘트
개선 제안
|
Beta Was this translation helpful? Give feedback.
-
참새 - Masonry Layout |
Beta Was this translation helpful? Give feedback.
-
여러 환경에서 잘 보이는가?
문서가 가독성 있게 쓰여있는가?
유연하게 재사용 하기 좋은가?
라벨 텍스트가 매우 긴 경우를 고려했는가?
정말 완벽한 사용성의 컴포넌트입니다!👍👍👏👏 |
Beta Was this translation helpful? Give feedback.
-
긍정적인 리뷰
개선제안
참새 므찌네요!!!👍 |
Beta Was this translation helpful? Give feedback.
-
긍정적인 리뷰전체적으로 컴포넌트가 깔끔했으며, 스토리북을 통해 컴포넌트를 설명하는 부분도 자세했습니다.
개선 제안
많이 배워갑니다 참새~ |
Beta Was this translation helpful? Give feedback.
-
1. 여러 환경에서 잘 보이는가
2. 문서가 가독성있게 쓰여있는가?
3. 유연하게 재사용하기 좋은가?
제안!Item 컴포넌트 하위에 Img 뿐만 아니라 텍스트도 고려하고 있어서 이 부분이 맞는 제안인지는 모르겠습니다. 그럼에도 img를 주로 사용한다는 가정하에 img 스타일에 *
https://developer.mozilla.org/ko/docs/Web/CSS/object-fit 추가정말 참새 Masonry 미쳤어요!!!! 완전 프로젝트에 써보고 싶을 정도에요 🙂 |
Beta Was this translation helpful? Give feedback.
-
여러 환경에서 잘 보이는가?반응형: 모바일 환경, PC 환경 모두 동등하게 잘 보입니다. 또한 itemSize를 퍼센트로 준다면 PC와 모바일 모두 다른 사이즈로 동일한 갯수로 보이도록 설정할 수 있다는 점이 인상 깊었습니다. 
문서가 가독성 있게 쓰여있는가?예시 코드가 스토리북에 잘 정리가 되어있고, Prop의 타입 또한 설명이 잘 되어 있어서 이해하기 편했습니다. 유연하게 재사용 하기 좋은가?내부의 children에서 사용자가 상호작용 코드를 추가로 작성할 수 있어서 너무 좋았습니다. 재사용성 최고! 라벨 텍스트가 매우 긴 경우를 고려했는가?텍스트의 경우에는 참새의 컴포넌트를 사용하는 개발자가 직접 처리를 하는 것이 적합하다고 생각이 들었고, 그렇게 답변도 들었습니다 긍정적인 리뷰PC, 모바일 모두 고려한 반응형이 잘 되는 점이 너무 좋았습니다. 그리고 사용자가 브라우저의 크기를 줄여도 느려지는 이슈 없이 정상 작동하는 점이 너무 좋았어요 개선 제안itmeSize 25% 와 같이 주었을 때 4개가 보여질 것을 예상했는데 3개가 떠서 24%로 줘서 사용했었어요. column 값을 추가해서 3개를 입력하면 3개를 보여주는 prop도 있으면 좋을 것 같아요. 이렇게 되었을 때 itemSize와 개수 2개가 있으면 문제가 생길 것 같아서요. 적고 보니 이대로 itemSize만 지원하는 것이 사용자에게 헷갈리지 않을 것 같다는 생각이 들었어요 단 한가지 개선점으로 적어볼 만한 것으로 row 방향으로도 지원해주면 더욱 좋을 것 같아요! |
Beta Was this translation helpful? Give feedback.
-
부엉이 - Tab Layout |
Beta Was this translation helpful? Give feedback.
-
|
className으로 지정을 해서 사용자에게 스타일을 위임을하고, 기능만 담당하는 레이아웃도 괜찮은 것 같습니다! 명시적으로 각 컴포넌트가하는 역할을 네이밍을 해주고, 컴포넌트 구조는 강제하고 스타일은 자유롭게 가져갈수 있는 것도 하나의 장점 같아요. 현재 스토리북에서도 할 수 있지만 defaultIdx를 설정을 두번째에 주고, 첫번째 컴포넌트에 1을 주어버리면 문제가 발생하네요! 또한 tab의 children 갯수가 다를 때(아래 예시), 사용자가 레이아웃 배치를 커스텀할 수 있도록 TabList의 스타일을 직접 커스텀 할 수 있게 하면 좋을 것 같습니다 :) 레포지토리에 있는 README 내용도 npm 에 있는대로 수정하면 좋을 것 같고, Tab, TabList등에도 이 컴포넌트가 무슨 일을 하는지 설명이 있으면 좋을 것 같아요 !! :D |
Beta Was this translation helpful? Give feedback.
-
긍정적인 리뷰
개선 제안
|
Beta Was this translation helpful? Give feedback.
-
리뷰🦉
개선 제안
|
Beta Was this translation helpful? Give feedback.
-
|
Beta Was this translation helpful? Give feedback.
-
긍정적인 리뷰
개선 제안
|
Beta Was this translation helpful? Give feedback.
-
패트릭 - Drawer |
Beta Was this translation helpful? Give feedback.
-
긍정적인 리뷰
개선제안
|
Beta Was this translation helpful? Give feedback.
-
긍정적인 리뷰
개선제안
|
Beta Was this translation helpful? Give feedback.
-
긍정적인 리뷰
개선 제안
|
Beta Was this translation helpful? Give feedback.
-
긍정적인 리뷰
개선제안
|
Beta Was this translation helpful? Give feedback.
-
팀 해온의 신나는 TabLayout 리뷰 시간 🥳https://hae-on.github.io/component-docs/docs/category/layout/ 긍정적 리뷰파인여러 스타일을 각각 설정할 수 있어 좋다. 참고할만한 좋은 설정이다 황펭children을 배열을 이용해서 여러 tab들을 만들 수 있게 명확하게 알려줘서 좋다. active, inactive 스타일을 커스텀할 수 있어 좋다. 타미사용자들이 쓸 때 쉽게 커스터마이징 할 수 있을 것 같아서 좋다. 루루탭 컴포넌트가 탭으로 움직일 수 있다는 것이 좋다. 적용해보고 싶다. 아커커스텀 자유도가 높아서 좋은 것 같다. 디자인도 예쁘다. 개선 사항 제안파인접근성을 고려해서 aria 속성들을 고려해보는 것이 어떤지 제안합니다. 황펭탭 이동할 때 애니메이션이 조금 있으면 좋을 것 같습니다. 애니메이션 선택까지 있다면 더 좋을 것 같습니다. 타미tabListStyles안에 activeStyle과 inactiveStyle이 있는데 문서에 계층을 분리해주면 좀더 좋을 것 같습니다. 루루탭과 container(탭과 내용)이 좀더 명확하게 연결되어있는 것이 보이면 좋겠습니다. 아커현재 모바일 환경에서 탭이 잘리게 보이는데 반응형을 좀 더 고려하면 좋을 것 같습니다. |
Beta Was this translation helpful? Give feedback.
-
|
좋네요.. |
Beta Was this translation helpful? Give feedback.
-
가브리엘🩳과 함께하는 SplitPane 리뷰!woowacourse/layout-component#62 긍정적 리뷰
개선 사항 제안
|
Beta Was this translation helpful? Give feedback.
-
제레미's Parallax-scrolling 리뷰👏 |
Beta Was this translation helpful? Give feedback.
-
긍정적인 부분
개선 제안
|
Beta Was this translation helpful? Give feedback.
-
긍정적인 리뷰예시코드를 npm readme에 써줘서 테스트해보기 편했습니다👍 개선제안<ParallaxScrolling perspective={0}>
<ParallaxItem
translateZ={-10}
scale={23}
imgUrl='https://d3ex8zhlkii399.cloudfront.net/galaxy.jpg'
objectFit='cover'
/>
<ParallaxItem
translateZ={-10}
scale={3}
imgUrl='https://d3ex8zhlkii399.cloudfront.net/neptune.png'
right='10000px'
objectFit='contain'
/>
<ParallaxItem
translateZ={-50}
scale={10}
imgUrl='https://d3ex8zhlkii399.cloudfront.net/mars.png'
objectFit='contain'
left='8000px'
/>
</ParallaxScrolling>배경과 같은 translateZ(-10)를 주면 같은 속도로 스크롤이 되고 배경보다 더 작은 값(-50)을 주면 배경이 이미지보다 느리게 스크롤이 되네여 |
Beta Was this translation helpful? Give feedback.
-
긍정적인 면
개선 제안
|
Beta Was this translation helpful? Give feedback.
-
|
긍정적인 부분 |
Beta Was this translation helpful? Give feedback.
-
긍정적인 부분
개선 제안
|
Beta Was this translation helpful? Give feedback.
-
솔로스타 - Tab Componentwoowacourse/layout-component#61 차린게 없습니다 |
Beta Was this translation helpful? Give feedback.
-
긍정적인 부분
개선 제안
|
Beta Was this translation helpful? Give feedback.
and others
-
테스트 및 리뷰할 때 아래 가이드라인을 참고해주세요.
Beta Was this translation helpful? Give feedback.
All reactions