Skip to content

Draw tab: Excel control order (Select, Eraser, pen set, Ink to Shape) - #1787

Closed
alosslessdev wants to merge 1 commit into
storytold:mainfrom
alosslessdev:draw/tab-control-order
Closed

alosslessdev wants to merge 1 commit into
storytold:mainfrom
alosslessdev:draw/tab-control-order

Conversation

@alosslessdev

Copy link
Copy Markdown
Contributor

Fixes #1783.

The Draw tab's controls now sit in Excel's order: Select, Eraser, the pen set, then Ink to Shape — and the object tool carries Excel's name and tooltip.

  • "Select Objects" becomes Select (same draw.lasso command; tooltip: select objects including ink, shapes and text areas).
  • The Eraser moves ahead of the pen set.

Labels, pen menus and the live stroke preview are unchanged; they are separate PRs (#1784, #1785, #1786). Cut from main, independent of #1767 and #1187.

Verification

  • cargo fmt --all -- --check
  • cargo clippy -p gridcraft-ui-egui --all-targets -- -D warnings
  • cargo test -p gridcraft-ui-egui (full crate suite)
  • Offscreen render (--example snapshot -- --tab Draw) shows Select, Eraser, Pen Blue/Black/Red, Highlighter and Ink to Shape in that order.
  • docs/ui-parity.md revision history updated; contributors/people.toml carries my own entry, per AGENTS.md.

@alosslessdev

Copy link
Copy Markdown
Contributor Author

Closing in favour of #1187, which replaces the Draw tab. #1783 is tracked on #1187. #1776 (Ink to Shape) stays open separately.

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.

Draw tab: control order matches Excel (Select, Eraser, pen set, Ink to Shape)

1 participant