[FEAT] [FE] DataTable 카테고리 셀 에디터 구현#585
Conversation
- columnId에 따라 분기 처리된 개별 셀 상태 추가 (textCell, categoryCell, amountCell, paymentCell) - 각 상태별 전용 Action 및 Reducer 로직 구현
Summary of ChangesHello @minngyuseong, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 Pull Request는 DataTable 컴포넌트의 셀 편집 기능을 구현하여 사용자가 테이블 내에서 직접 데이터를 수정할 수 있도록 합니다. 각 셀의 데이터 타입에 따라 적절한 에디터를 제공하며, 상태 관리를 통해 편집 UI를 효율적으로 제어합니다. 이를 통해 데이터 관리의 편의성을 높이고 사용자 경험을 개선합니다. Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이번 PR은 데이터 테이블에 셀 편집 기능을 추가하는 멋진 기능입니다. 구현 과정에서 리듀서를 사용하여 여러 에디터 상태를 관리하는 방식은 좋습니다. 하지만, 새로 추가된 에디터 컴포넌트들에서 몇몇 중대한 복사-붙여넣기 오류를 발견했습니다. 이 오류들은 기능이 의도대로 동작하지 않게 할 수 있습니다. 코드 중복에 대한 부분은 현재 임시 구현임을 고려하여 향후 개선 사항으로 제안합니다. 자세한 내용은 각 리뷰 코멘트를 확인해주세요.
minngyuseong
changed the title
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
1jiwoo27
left a comment
1jiwoo27
left a comment
There was a problem hiding this comment.
수고하셨습니다! 카테고리 클릭 했을 때 Editor 잘 열리네요 🤩
TextCellEditor나 CategoryCellEditor 클릭 했을 때, 열이 옆으로 밀리고 있는데 w 지정해서 안 밀리게 해주실 수 있나요?
Kjiw0n
left a comment
Kjiw0n
left a comment
There was a problem hiding this comment.
코멘트 및 질문 확인 부탁드립니다!
- home-page/, landing-page/에
columns가 2개 있는 이유가 무엇인가요? CategoryCellEditor,TextCellEditor에 공통 로직의 중복이 확인되고있어요. 결제내역 등도 추가될 것 같은데, 하나의 공통 컴포넌트로 뽑아주면 좋을 것 같습니다!CategoryCellEditor와CategoryFilter가 같은 UI이고, 다음의 차이가 있는 것 같아요. 하나의 컴포넌트로 해결할 수 있다면 좋을 것 같습니다. (결제수단Filter, CellEditor도!)- 단일선택/중복선택
- 실행트리거 위치
minngyuseong
force-pushed
the
feat/#579/context-menu
branch
from
cd9c617 to
beb6010
Compare
|
제목 [FE] [FEAT] 로 바꿔주세요! |
1jiwoo27
left a comment
1jiwoo27
left a comment
There was a problem hiding this comment.
수고하셨습니다 ❗
선택한 카테고리가 셀에 반영되는 거는 API 연동하면서 하시는 건가요?!
넹! 너무 길어용 |
3 participants
관련 이슈
closes #579
작업 내용
테이블 행에서 '카테고리'바로 수정 가능하도록 context-menu 구현
셀에디터 공통 Anchor 컴포넌트화 (셀위치, 스크롤 막기)
필터 및 셀에디터에서 공통으로 사용되는 리스트 컴포넌트화
키보드 방향키 제어 로직을 검색 훅에서 분리 및 셀에디터에서 사용
스크린샷
💬 공유사항 to 리뷰어
DataTableFilter에서 방향키로 옵션 선택하는 로직을 훅으로 분리했어요.
필수 체크사항