22.5: Bind the remaining CS6 keymap shortcuts

オープン
  • 何か問題がありますか？
#79 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
48/100
issue の種類
機能追加
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
rust
領域
desktop, devtools

調査の方向性

Read docs/research/fw-ui-evidence.md for the shortcut encoding, then compare the CS6 shortcut data with the menu rows and SHORTCUTS table in crates/app/src/menus.rs. Start by identifying rows without accelerators and checking modifier mappings; keep any Adobe source bytes out of commits. Done means each new binding matches in both places, the two named tests pass, no modifier+key is duplicated, and the bound and unbound counts plus reasons are documented.

索引モデルが issue の本文から書いたものです。

説明

enhancement parity

Phase 22.5 (CS6 shell fidelity): the shell binds only part of the CS6 keymap, so muscle memory from Fireworks CS6 does not carry over.

Source of truth

  • source/Adobe Fireworks CS6/English/Keyboard Shortcuts/Fireworks.xml — the CS6 shortcut set (<command id="cmd_ZoomIn"><shortcut text="4 187"/></command>). The mods bitfield is 4=Ctrl, 1=Alt, 2=Shift; the second number is a Windows virtual-key code. The encoding is recorded in docs/research/fw-ui-evidence.md.
  • source/ is git-ignored analysis input. No Adobe file bytes may enter a commit — only derived data.

Work

  • Decode the CS6 shortcut set and join it to the menu rows in crates/app/src/menus.rs (each row carries a CS6 resource id, a label and an accelerator string).
  • Bind the rows that currently have no accelerator, updating both the row's accel string and the SHORTCUTS table so the key actually fires.
  • Correct the modifier mismatches: CS6's mods bitfield 5 is Ctrl+Alt, and rows declared Ctrl+Shift where CS6 says Ctrl+Alt must change.
  • Where a row genuinely cannot be bound, leave it unbound and record the reason.

Acceptance

  • Every newly bound row has both an accel string and a matching SHORTCUTS entry, and the existing shortcuts_cover_the_core_map / every_shortcut_has_a_help_name tests still hold.
  • No two commands are bound to the same modifier+key.
  • The count of bound vs still-unbound rows is documented, with a reason per remaining gap.

Tracking: tasks #204, #169, #186.

主要言語
Rust
スター
11
フォーク
1
平均マージ
9時間 48分
マージ済み PR（30日）
13

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

aliasfoxkde/FireCraft のほかの issue

aliasfoxkde/FireCraft の issue をすべて見る

似ている issue

Rust の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。