Skip to content

[MOBTECH-787] Semantic Token 변경사항 반영#115

Merged
bonoogi merged 2 commits into
developfrom
feature/change-semantic-tokens
Jan 14, 2026
Merged

[MOBTECH-787] Semantic Token 변경사항 반영#115
bonoogi merged 2 commits into
developfrom
feature/change-semantic-tokens

Conversation

@bonoogi

@bonoogi bonoogi commented Jan 14, 2026

Copy link
Copy Markdown
Contributor

기한

  • exp머지:
  • 배포일:
  • 리뷰 종료 시점 : 1/16(금)
  • 하드 듀 : 따로 없음

요약

  • 피그마 베지어 v3 Color 테이블에 맞춰서 변경사항 적용
  • fillGreyLight 토큰 추가 (Light: grey25, Dark: grey900)
  • 피그마 베지어 v3 Color 테이블에 없는 12개 토큰 제거 (gradientAccentGreenHeavy, surfaceAccent* 11개)

세부적인 작업 내용

추가된 토큰 (1개)

  • fillGreyLight: Light → grey25, Dark → grey900

제거된 토큰 (12개)

  • gradientAccentGreenHeavy
  • surfaceAccentBlue
  • surfaceAccentCobalt
  • surfaceAccentGreen
  • surfaceAccentNavy
  • surfaceAccentOlive
  • surfaceAccentOrange
  • surfaceAccentPink
  • surfaceAccentPurple
  • surfaceAccentRed
  • surfaceAccentTeal
  • surfaceAccentYellow

참고 링크

  • 리니어:
  • 노션:
  • 피그마:
  • 채널톡:

셀프 체크

  • 리니어 티켓을 첨부하였습니다
  • 코딩 컨벤션 에 맞춰서 작성했습니다
  • 리뷰 리퀘스트 전에 셀프 리뷰를 진행했습니다
  • 기한을 작성하였습니다.

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 새로운 라이트 그레이 색상이 디자인 시스템에 추가되었습니다.
  • 변경 사항

    • 여러 서피스 악센트 색상(파랑, 코발트, 초록, 네이비, 올리브, 주황, 핑크, 보라, 빨강, 틸, 노랑)과 그린 그래디언트가 팔레트에서 제거되었습니다.
    • 텍스트 스타일 지정 시 색상을 명시적으로 지정해야 합니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@channeltalk

channeltalk Bot commented Jan 14, 2026

Copy link
Copy Markdown

@coderabbitai

coderabbitai Bot commented Jan 14, 2026

Copy link
Copy Markdown

개요

이 변경 사항은 의미론적 색상 토큰에서 11개의 표면 강조 색상 케이스와 그래디언트 케이스를 제거하고, 새로운 fillGreyLight 케이스를 추가합니다. 또한 타이포그래피 메서드에서 semanticColor 매개변수의 기본값을 제거하여 호출자가 명시적으로 제공하도록 변경합니다.

변경 사항

응집 / 파일 변경 요약
의미론적 토큰 업데이트
Sources/BezierSwift/Foundation/Color/V3/BCSemanticToken.swift
11개의 surfaceAccent* 케이스(Blue, Cobalt, Green, Navy, Olive, Orange, Pink, Purple, Red, Teal, Yellow) 및 gradientAccentGreenHeavy 제거. fillGreyLight 케이스 추가. 팔레트 매핑 확장 업데이트.
타이포그래피 API 변경
Sources/BezierSwift/Foundation/Typography/Typography.swift
BezierFont.attributedString() 메서드에서 semanticColor 매개변수의 기본값(.txtBlackDarkest) 제거. 호출자가 명시적으로 색상을 지정해야 함.

예상 코드 리뷰 난이도

🎯 2 (Simple) | ⏱️ ~12분

토끼의 축하시

🐰 토큰 정리하고, 회색빛 더하니
강조색은 가고, 명시성은 오네!
색상의 정원을 가꾸는 변화,
더욱 깔끔하고 명확해진 API!

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 Semantic Token 변경사항을 반영한다는 주요 내용을 명확하게 요약하고 있으며, raw_summary의 BCSemanticToken 파일 변경사항과 직접 연관이 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Comment @coderabbitai help to get the list of available commands and usage tips.

@bonoogi bonoogi changed the title [ETC] Semantic Token 변경사항 반영 [MOBTECH-787] Semantic Token 변경사항 반영 Jan 14, 2026
Comment on lines -183 to -193
case surfaceAccentBlue
case surfaceAccentCobalt
case surfaceAccentGreen
case surfaceAccentNavy
case surfaceAccentOlive
case surfaceAccentOrange
case surfaceAccentPink
case surfaceAccentPurple
case surfaceAccentRed
case surfaceAccentTeal
case surfaceAccentYellow

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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


// MARK: - Gradient
case gradientAccentGreen
case gradientAccentGreenHeavy

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

삭제 관련 맥락

요약하자면 gradientAccentGreenHeavy 토큰의 값이 이상해서 수정하던 와중에 컴케를 해보니 light만 있으면 되는거였음(gradientAccentGreenHeavy는 사용처 없음)

public func attributedString(
_ component: BezierComponentable,
text: String,
semanticColor: SemanticColor = .txtBlackDarkest,

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

신버전 토큰을 받는 메서드와 구버전 토큰을 받는 메서드 둘 다 기본값을 받다보니 시그니처가 동일해보이는 문제가 있어서 구버전 토큰을 받는 쪽은 기본값을 안받도록 해두었습니다

@solchan87 solchan87 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

감사합니다!

@bonoogi
bonoogi merged commit ba91216 into develop Jan 14, 2026
1 check passed
@bonoogi
bonoogi deleted the feature/change-semantic-tokens branch May 18, 2026 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants