Skip to content
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.

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

tiptap 노드 및 마크 전체 리팩토링 #1111

Merged
merged 1 commit into from
Jan 30, 2024

Conversation

devunt
Copy link
Member

@devunt devunt commented Jan 22, 2024

팁탭 노드 및 마크 등 관련 코드 전체 에디터 리뉴얼에 맞게 리팩토링함

주요 변경사항

  • prose.css 파일을 통해 스타일링하던 부분들을 이제 각 노드/마크/익스텐션에서 바로 스타일을 적용하도록 함
  • letter_spacing, line_height, text_align 익스텐션을 deprecate하고 paragraph의 attribute로 옮김 (heading이 사라져서 paragraph에만 적용하면 됨)
  • font_family 익스텐션을 deprecate하고 font_family 마크로 재구현함 (이제 인라인으로 적용 가능)
  • horizontalRule 노드를 deprecate하고 horizontal_rule 노드로 재구현함 (네이밍 이슈)
  • heading 노드를 deprecate함 (이제 paragraph 노드로 처리 가능)
  • paragraphlevel attribute를 deprecate함 (이제 폰트 크기를 임의로 설정 가능)
  • text_color 마크를 deprecate하고 font_color 마크로 재구현함 (이제 프리셋 컬러 이름이 아닌 hex code를 입력받음)
  • font_size 마크를 신규로 추가함 (이제 폰트 크기를 인라인으로 변경 가능)
  • 기타 자잘한 코드 수정 및 개선

Copy link

graphite-app bot commented Jan 22, 2024

Your org requires the Graphite merge queue for merging into main

Add the label “:rocket:” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

Copy link
Member Author

devunt commented Jan 22, 2024

Current dependencies on/for this PR:

This stack of pull requests is managed by Graphite.

@mu-hun mu-hun added the preview 👀👀👀 label Jan 22, 2024
@withglyph
Copy link

withglyph bot commented Jan 22, 2024

배포가 끝나면 아래 주소에서 프리뷰를 확인할 수 있어요.

프로젝트 프리뷰 URL
penxle.com https://penxle-pr-1111.pnxl.site
penxle.io https://company-pr-1111.pnxl.site
help.penxle.com https://help-pr-1111.pnxl.site

@devunt devunt force-pushed the feature/tiptap-styling branch 4 times, most recently from 5dc7e5b to d40821d Compare January 23, 2024 14:37
@devunt devunt changed the base branch from feature/tiptap-embed to main January 24, 2024 05:21
@devunt devunt force-pushed the feature/tiptap-styling branch 5 times, most recently from ae7d7c6 to 26638b5 Compare January 24, 2024 15:00
@devunt devunt added the 🚀 #가보자고 label Jan 24, 2024 — with Graphite App
@devunt devunt force-pushed the feature/tiptap-styling branch 4 times, most recently from 1974b9d to 6ca5c8e Compare January 25, 2024 05:52
@devunt devunt changed the title tiptap 노드 및 마크 스타일링 방식 개선 tiptap 노드 및 마크 전체 리팩토링 Jan 25, 2024
@devunt devunt force-pushed the feature/tiptap-styling branch 2 times, most recently from 0b0118d to 4511335 Compare January 25, 2024 06:18
@devunt devunt marked this pull request as ready for review January 25, 2024 06:29
mu-hun
mu-hun previously requested changes Jan 26, 2024
Copy link
Contributor

@mu-hun mu-hun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

문단 행간, 자간, 정렬 레거시 지원이 인라인스타일로 넣는 기본값에 명시도가 가려져서 하위호환 검수가 더 필요합니다...

기존 이번 수정

@devunt devunt force-pushed the feature/tiptap-styling branch 2 times, most recently from ce7c8f8 to 24485da Compare January 27, 2024 14:13
@devunt devunt changed the base branch from main to deps January 27, 2024 14:13
@devunt devunt requested a review from mu-hun January 29, 2024 05:03
Copy link

graphite-app bot commented Jan 30, 2024

Merge activity

팁탭 노드 및 마크 등 관련 코드 전체 에디터 리뉴얼에 맞게 리팩토링함

**주요 변경사항**

- `prose.css` 파일을 통해 스타일링하던 부분들을 이제 각 노드/마크/익스텐션에서 바로 스타일을 적용하도록 함
- `letter_spacing`, `line_height`, `text_align` 익스텐션을 deprecate하고 `paragraph`의 attribute로 옮김 (`heading`이 사라져서 `paragraph`에만 적용하면 됨)
- `font_family` 익스텐션을 deprecate하고 `font_family` 마크로 재구현함 (이제 인라인으로 적용 가능)
- `horizontalRule` 노드를 deprecate하고 `horizontal_rule` 노드로 재구현함 (네이밍 이슈)
- `heading` 노드를 deprecate함 (이제 `paragraph` 노드로 처리 가능)
- `paragraph`의 `level` attribute를 deprecate함 (이제 폰트 크기를 임의로 설정 가능)
- `text_color` 마크를 deprecate하고 `font_color` 마크로 재구현함 (이제 프리셋 컬러 이름이 아닌 hex code를 입력받음)
- `font_size` 마크를 신규로 추가함 (이제 폰트 크기를 인라인으로 변경 가능)
- 기타 자잘한 코드 수정 및 개선
@devunt devunt changed the base branch from deps to main January 30, 2024 05:34
@graphite-app graphite-app bot merged commit 9f95331 into main Jan 30, 2024
9 checks passed
@graphite-app graphite-app bot deleted the feature/tiptap-styling branch January 30, 2024 05:35
@withglyph withglyph bot removed the preview 👀👀👀 label Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 #가보자고
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants