[pierrevbc]

Problem

A right click on an unselected shape opens the slide's context menu (Layout, Reset Slide, Format
Background...), not the shape's: the shape has to be left-clicked first. In PowerPoint, the
right-button press selects what's under the pointer, then the menu matches it.

Reproduce on main (0.6.0): deckcraft --sample, right-click the title of slide 1 without
selecting it first: slide menu, title not selected.

Cause: the canvas only sends the primary button to Session::pointer
(crates/ui-egui/src/canvas.rs, pointer), and context_menu picks the shape or slide menu from
the current selection.

Expected (as in PowerPoint):

• unselected shape: selected alone (a text box or placeholder is not put into editing, nothing is
  dragged), shape menu;
• shape already selected, multiple selection included: selection kept;
• empty slide: selection cleared, slide menu;
• inside the text being edited: caret and text selection kept, and the menu acts on the text.

What changes

Two commits: the change, and the progress docs.

• Engine. New PointerKind::ContextClick (contextclick, or rightclick, in ui.pointer
  and the MCP pointer tool). It selects as listed above and returns {hit, selected, editing}.
  It never edits text, starts a drag or adds an undo step, and does nothing while a drawing tool
  or a gesture is active. In the box being edited it uses the same zone as a left press that
  places the caret (the box's bounds inflated by the hit tolerance).
• Canvas. The secondary-button press goes through Session::pointer before egui opens the
  context menu (on release). While text is being edited, the menu shows text items: Cut, Copy and
  Paste (which already act on the selected text in text mode), Paragraph..., Bullets, Numbering,
  Link..., Format Shape....
• docs/control-protocol.md lists the new pointer kind; the MCP pointer tool describes it.
• Progress docs. The context-menu row of docs/ui-parity.md (status line, history) and a
  ROADMAP.md progress log line. No checklist row changes score and no gaps.md entry covered
  this.

How it was tested

cargo xtask ci passes on this branch (fmt, clippy -D warnings, workspace tests, layers,
assets, wasm32 check).

New tests in crates/engine/src/tests.rs, on decks built in code:

• right_click_selects_the_shape_under_the_pointer: unselected shape, text box (selected, not
  edited), multiple selection kept, empty slide cleared, no drag, no undo step.
• right_click_in_edited_text_keeps_the_text_selection: text selection kept; Copy and Cut act on
  it; right click on another shape leaves editing and selects it.

Checked by hand in the app, on 0.5.0 with this change and the frame-drag one together
(deckcraft --sample --control, real egui pointer events, ui.screenshot): right click on an unselected subtitle selects it and shows the shape menu; right
click in edited text keeps the selected word and shows the text menu; right click on empty slide
shows the slide menu.

Known limits

• A shape inside a group: right click selects the group, as a left click does today.
• The text items of the menu are the common ones; PowerPoint's text menu is longer.

Related PRs

• The PR "Canvas: drag the frame of a text box being edited to move it" is independent. Whichever
  lands second needs one adjustment: context_click should use that PR's edited_box/edit_zone
  for the edited-text test and list Hit::Frame among the hits that carry a shape id (the
  compiler flags the match). I will rebase the second one once the first lands. The adjustment
  is already written and tested in a local integration of both: a right click anywhere in the
  edit zone, frame and rotated boxes included, keeps the caret and the text menu.
• i18n: add Polish (pl) interface #199, i18n: add Russian, Spanish, French, Portuguese and German interfaces #110 and Add Ukrainian interface translation and language selection #34 (interface translations) change the context menu strings in
  crates/ui-egui/src/canvas.rs: textual conflict in context_menu; the new text items would
  then go through their translation function.
• Pictures: Change Picture replaces the selected picture (#115) #204, Animation effects: follow the system's reduced-motion setting #196, Charts: GridCraft spreadsheet data editor and live editing for the first 5 chart types #195 and Issue #106: zooming keeps the point under the pointer #156 also change canvas.rs; Export: animated GIF of the deck (Export… dialog, CLI, MCP) (#171) #177 crates/mcp/src/tools.rs; Animation effects: follow the system's reduced-motion setting #196, Slide show ink: pen, highlighter, eraser, laser, keep-on-exit, ink select tool #161,
  feat(theme): add System appearance across DeckCraft controls #134 crates/ui-egui/src/control.rs and docs/control-protocol.md: no textual conflict.
• Slides: Ctrl+A in the slide pane selects every slide (#188) #191, Notes: speaker notes show their formatting and keep it when edited #168, Charts: GridCraft spreadsheet data editor and live editing for the first 5 chart types #195 add tests at the end of crates/engine/src/tests.rs: append-only conflict.
• docs/ui-parity.md and ROADMAP.md (status line, progress log) conflict with other PRs that
  update them (Add Ukrainian interface translation and language selection #34, i18n: add Russian, Spanish, French, Portuguese and German interfaces #110, feat(theme): add System appearance across DeckCraft controls #134, Text: typing goes on after Tab in a text box (#78) #139, Export: animated GIF of the deck (Export… dialog, CLI, MCP) (#171) #177, Charts: GridCraft spreadsheet data editor and live editing for the first 5 chart types #195, Animation effects: follow the system's reduced-motion setting #196, i18n: add Polish (pl) interface #199); the resolution is to keep both lines.

[echelon]

Thank you for this contribution, @pierrevbc! It has landed on main and will ship in the next DeckCraft release.

If you haven't already, please come join us on Discord: https://discord.gg/artcraft