22.5: Bind the remaining CS6 keymap shortcuts

Abierto
  • ¿Algo no está bien?
#79 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
48/100
Tipo de issue
Nueva funcionalidad
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
rust
Área
desktop, devtools

Línea de trabajo

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.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.

Lenguaje dominante
Rust
Estrellas
11
Forks
1
Merge medio
9 h 48 min
PR fusionados (30 d)
13

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de aliasfoxkde/FireCraft

Todos los issues de aliasfoxkde/FireCraft

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.