Skip to content

Conversation

@vividtone
Copy link
Contributor

対応方針:

  • 本テーマによる色の再定義は可能な範囲で削除しデフォルトテーマの色を使用
  • 本テーマで再定義している色は可能な範囲でOpen Colorの色に変更

変更内容:

  • body に対する color 宣言を削除。デフォルトテーマの oc-gray-9 を使用
  • headings に対する color 宣言の値 #222 をOpen Colorの近似色 oc-gray-9 に置換
  • #sidebar h3 に対する color 宣言を削除。デフォルトテーマの oc-gray-7 を使用
  • #watchers a.delete に対する opacity 宣言を削除。デフォルトテーマでは opacity が削除されたため
  • div.issue に対する color 宣言を削除。デフォルトテーマの oc-yellow-0 を使用
  • .issue .contextualbackground 宣言を半透明の白を使うよう調整。これまでの背景色はテーマで再定義したチケットの背景色 #ffffe6 との調和を考慮して #fffff2 を使用していたが、チケットの背景色としてデフォルトテーマの背景色 oc-yellow-0 が使われるようになったため、 oc-yellow-0 との調和を考慮
  • #222 をOpen Colorの近似色 oc-gray-9 に変更
  • #444 をOpen Colorの近似色 oc-gray-7 に変更
  • #484848 をOpen Colorの近似色 oc-gray-7 に変更
  • #BBBBBB をOpen Colorの近似色 oc-gray-4 に変更
  • #ccc をOpen Colorの近似色 oc-gray-4 に変更

- 本テーマによる色の再定義は可能な範囲で削除しデフォルトテーマの色を使用
- 本テーマで再定義している色は可能な範囲でOpen Colorの色に変更
Copy link
Collaborator

@ishikawa999 ishikawa999 left a comment

Choose a reason for hiding this comment

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

色が変わっている箇所の確認と、open-colorが導入される前のRedmineで動かしたときもプロパティが存在しない時用の代替値によって問題なく表示されることを確認しました

Copy link
Contributor

@hidakatsuya hidakatsuya left a comment

Choose a reason for hiding this comment

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

差分だけ見ました。LGTM

*
* opacity: 0.4 -> 0.7;
* Example:
* color: var(--oc-gray-9, #212529);
Copy link
Contributor

Choose a reason for hiding this comment

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

👍🏼

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.

4 participants