Skip to content

에디터 및 상수 리팩토링 / codeSyntaxHighlighter 코드 복사 구현 및 스타일 수정 (main) #355

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
Ryan-TheLion merged 4 commits into from
Jun 3, 2024
Merged

에디터 및 상수 리팩토링 / codeSyntaxHighlighter 코드 복사 구현 및 스타일 수정 (main) #355

Ryan-TheLion merged 4 commits into from
Jun 3, 2024

Conversation

@Ryan-TheLion
Copy link
Collaborator

@Ryan-TheLion Ryan-TheLion commented Jun 3, 2024

관련 이슈

close: #351

개요

에디터 리팩토링, codeSyntaxHighlight 코드 복사 및 스타일 수정

상세 내용

에디터

  • 동일 역할을 하는 토스트 에디터, 뷰어 컴포넌트를 하나의 컴포넌트(ContentEditor, ContentViewer) 로 재사용 하는 구조로 수정
  • 토스트 에디터 업로드 이미지 훅을 하나로 재사용하는 구조로 수정(이로 인해 불필요하다고 생각되는 기존 리코일 상태 관련 파일 삭제)
  • code syntax highlighter 스타일 변경 및 코드 복사 기능 구현

code-syntax-highlight

상수

  • 하나의 파일에 리액트 쿼리 관련 훅을 여러개 작성하는 구조에서, 필요한 리액트 쿼리를 별도의 파일로 분리하고 관련 폴더에 사용하는 구조로 수정(수정 이후 기존 리액트 쿼리 커스텀 훅 파일 삭제)
  • 리액트 쿼리 키를 atomic 하게 조합해서 사용한다고 생각되어, 각 상황에 맞는 변수 및 카테고리로 분리(조금 더 재사용할 수 있다고 생각되는 구조로 수정)
  • 사용되지 않는 불필요한 상수 삭제

커스텀 훅

  • 하나의 커스텀 훅에 너무 많은 것을 정의하고 있다는 생각이 들어, 필요하다고 생각되는 부분(api 등)을 제외하고 나머지 제거 및 코드 리팩토링

모각코

  • 모각코 상세 목록보기 스타일을 기존 커피챗 상세 등에서 사용하는 스타일로 수정(LinkToPage 재사용)

coding-meeting-back

@Ryan-TheLion
refactor: 내 답변
35d61c7 
- ContentEditor를 재사용하여 내 답변 에디터를 렌더링
- react-query에 있던 일부 query 훅을 별도로 분리
(이후에도 분리하여 최종적으로 관련된 곳에서 별도의 파일로
관리하는 것으로 리팩토링 할 예정)
- 업로드된 이미지 관련 컴포넌트를 최대한 공용으로
사용할 수 있도록 구현해보고, 내 답변 에디터에 재사용하여 구현
- queryKey 리팩토링 중(atomic => 좀 더 직관적으로)
@Ryan-TheLion
refactor: 에디터, 뷰어 공용 컴포넌트로 재사용
a5d2bd2 
- 기존 동일 역할을 하는 에디터, 뷰어 컴포넌트가
복붙형태로 사용되던 형태를 하나의 에디터 뷰어로
재사용하는 구조로 수정
- 에디터 이미지 업로드 훅도 하나로 재사용하는
구조로 수정
- 커스텀 훅에 필요하다고 생각되는 기능외에
불필요하다고 생각되는 부분은 삭제
- 하나의 파일에 리액트쿼리를 한번에 관리하던 구조에서
필요한 곳에서 별도로 커스텀훅 파일로 관리하는 구조로 변경
- 코드 하이라이터 스타일 변경 및 코드 복사 기능 구현
- 리액트 쿼리 키 상수를 보다 atomic하게 조합하던 구조에서
조금 더 명시적으로 사용하는 구조로 수정
@Ryan-TheLion
refactor: limitation, validate 상수
8737dfc 
- limitation 상수: 카테고리로 분류,
기존 Limitation 상수 삭제
- valitate 상수: 사용되지 않는 불필요한
상수 제거, 카테고리 분류
- 버튼 third 스타일 수정
- 모각코 상세 목록보기 버튼을
LinkToPage 컴포넌트로 교체(재사용)
@Ryan-TheLion
Merge pull request #354 from KernelSquare/feat/Frontend#351
ddbc254 
에디터 및 상수 리팩토링 / codeSyntaxHighlighter 코드 복사 구현 및 스타일 수정
@Ryan-TheLion Ryan-TheLion added Feature 기능개발 Refactor 리팩토링 design CSS 등 디자인 관련 변경 labels Jun 3, 2024
@Ryan-TheLion Ryan-TheLion self-assigned this Jun 3, 2024
@Ryan-TheLion Ryan-TheLion merged commit a318442 into main Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@Ryan-TheLion Ryan-TheLion

Labels

design CSS 등 디자인 관련 변경 Feature 기능개발 Refactor 리팩토링

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

에디터, 업로드 훅 공용으로 통합 / Limitation 리팩토링

2 participants

@Ryan-TheLion