Repository navigation
Conversation
ミュートボタンは「新着で音が鳴る、アカウントごとのストリーム系カラム」に しか無かった。リアクションしたときの音はノートを表示するすべてのカラムで 鳴るので、全アカウントの各カラムや検索・お気に入り・クリップ・ユーザー・ 照会・みつけるでは、鳴るのに止める手段がヘッダーに無かった。 - ノートを描くカラムの DeckColumn すべてに sound-enabled を付ける - DeckNoteColumn は必ずノートを描くので、呼ぶ側の指定をやめて常に付ける - ノートを描くのにボタンの指定が無いカラムを lint で落とす Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
アカウントごとの TL は新着で音が鳴るが、全アカウントの TL / メンション / リスト等は鳴らなかった。全アカウント用の取得処理に、新着の音を鳴らす 箇所が無かった (全アカウントの通知カラムは鳴る)。 アカウントごとの TL と同じ規則で鳴らす: 画面の先頭に差し込まれた新着で 鳴らし、自分のノートは自分のノート用の音にし、カラムをミュートして いたら鳴らさない。カラムにアカウントが無いので、音の出所は届いた ノートの取得元アカウントのサーバーにする。 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…note-columns-1388 fix(deck): 音が鳴るすべてのカラムに音声ミュートボタンを出し、全アカウントのカラムでも新着で鳴らす (#1388)
プレビューのポップアップは、表示名には絵文字の対応表 (ユーザー情報に 付いてくる、絵文字名 → 画像の URL) を渡していたが、自己紹介には渡して いなかった。リモートユーザーの絵文字は相手のサーバーのものなので、 渡さないと見ている側のサーバーの辞書で引いて見つからない。 プロフィールのウィンドウは自己紹介にも渡している。同じ形に揃える。 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
MFM エディターの入力欄は素のテキスト入力欄で、入れ子になった MFM を 追いにくかった。 - 関数 ($[名前 と引数)、装飾の記号、カスタム絵文字、メンション、 ハッシュタグ、URL、コードに色を付ける - $[名前 と対応する ] を、入れ子の深さごとに同じ色にする - カーソルを置いた括弧の相手を示す (対象は関数の [ ] だけ) ソースの文字は変えない。自動の字下げも括弧の自動補完も入れない: MFM は 空白と改行をそのまま表示するので、エディターが文字を足すと表示が変わる。 同じ理由で、ソースを整形する形のインデントは採らない。 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…-emojis-1386 fix(user): ユーザーのプレビューで、リモートユーザーの自己紹介のカスタム絵文字を出す (#1386)
DEVELOPMENT.md には、環境の準備と日々の作業、コーディング規約、各部の 設計の解説が同居していて、外部の人は入口を、内部の開発者は目当ての 設計を探しにくかった。文章は書き換えず、置き場だけを変える。 - CONTRIBUTING.md: 外部の人の入口。最短の手順と、やりたいこと別の案内 - DEVELOPMENT.md: 準備 / コマンド / テスト / ビルド / コーディング規約 / フォーク対応の手順。冒頭に文書の地図を置く - AI.md (新設): AI チャット、手元の CLI、人格と記憶、capability、 資格情報、HEARTBEAT - DECK.md (新設): ウィンドウとカラム、ノートの一覧、カラムクエリ、 配布物、テーマ、ゲスト表示 - ARCHITECTURE.md: クレートとプロセスの構成、起動の流れ、OS の状態への 適応を移す。元からあった「目指す構成」の要約と題が重なるので、移した 側を「目指す構成の詳細」と改めて相互にリンクする - PERFORMANCE.md: パフォーマンス設定とレンダリングの規約を移す ほかの文書・コードのコメント・lint の説明からの参照を、移した先に 合わせる。 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…1392 feat(mfm): MFM エディターの入力欄に色を付け、入れ子の括弧を深さで色分けする (#1392)
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- PERFORMANCE.md: Jank は「33ms 超」固定ではなく時間予算の 2 倍超。下げる閾値と 上げるまでの秒数は設定で変わるので数値を書かず、正本のファイルを指す - DECK.md: 参照定義の無い [DESIGN.md] をリンクにする (移す前からあった) - CONTRIBUTING.md: 「最短の手順」が本家の clone なので、フォークを clone すると明記 Refs #1389 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
CSS レンダリング規約を PERFORMANCE.md へ移したときの付け替え漏れ。 Refs #1389 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
docs: DEVELOPMENT.md を手引きと規約に絞り、設計の解説を分野ごとの文書へ分ける (#1389)
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 34 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. |
Deploying notedeck with
|
| Latest commit: |
5879a04
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://82d4bac8.notedeck-d3a.pages.dev |
| Branch Preview URL: | https://develop.notedeck-d3a.pages.dev |
API surface diff外部アプリ向け API 面 ( Full diffdiff --git a/src-tauri/openapi.json b/src-tauri/openapi.json
index 4996cb3..b44b880 100644
--- a/src-tauri/openapi.json
+++ b/src-tauri/openapi.json
@@ -6,7 +6,7 @@
"license": {
"name": "MIT"
},
- "version": "1.107.0"
+ "version": "1.108.0"
},
"paths": {
"/api": {
|
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. |
1 similar comment
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
変更
追加・変更
$[...])・絵文字・メンション・ハッシュタグ・URL・コードに色を付け、入れ子の括弧を深さで色分けする。ソースの文字は変えない (整形や字下げはしない)。入力欄の背景はコード用の面になる修正
文書
DEVELOPMEND.mdの肥大化とCONTRIBUTING.mdとの役割分担 #1389): 新設AI.md/DECK.md、ARCHITECTURE.md / PERFORMANCE.md / CONTRIBUTING.md へ移動。文章は置き場を変えただけで、レビューで見つかった古い記述 (Jank の判定条件) だけ実装に合わせた確認の状態
Closes #1388
Closes #1386
Closes #1392
Closes #1389
🤖 Generated with Claude Code