update drawing / editing mode icons

[PythonFZ]

Summary by CodeRabbit

Release Notes

• Style
  • Updated the icon for the drawing mode toggle button to improve visual clarity.
  • Enhanced drawing mode toggle tooltips with keyboard shortcut indicators.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

A single file update that replaces the EditIcon with OpenWithIcon in the landing page and appends "(X)" mnemonic shortcuts to the drawing mode enable/disable tooltips.

Changes

Cohort / File(s)	Summary
Icon and Tooltip Updates app/src/pages/landingPage.tsx	Replaced EditIcon import with OpenWithIcon; updated all icon usages; appended "(X)" mnemonic to "Enable drawing mode" and "Disable drawing mode" tooltip strings

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 The icon takes a turn so new,
From edit to open—a fresh view!
With "X" mnemonics dancing bright,
Drawing modes now guide us right!
A hop, a skip, and we're drawing free! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title accurately describes the main change: replacing drawing/editing mode icons (EditIcon with OpenWithIcon) and updating related tooltips.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between aaeda38 and 3d6a326.

📒 Files selected for processing (1)

• app/src/pages/landingPage.tsx

🔇 Additional comments (3)

app/src/pages/landingPage.tsx (3)

18-18: LGTM!

The import change from EditIcon to OpenWithIcon is appropriate. The OpenWithIcon (four-directional arrow) better represents editing/manipulation mode semantically than a pencil icon.

---

301-324: Good icon choice for editing mode.

The OpenWithIcon (move/drag icon with four directional arrows) is semantically more appropriate for editing mode, which typically involves repositioning or manipulating elements. This is a meaningful improvement over the generic EditIcon.

---

276-300: Keyboard shortcut verified. The "(X)" mnemonic in the tooltip correctly matches the keyboard binding in KeyboardShortcutsHandler.tsx (lines 289-300), where pressing x/X toggles drawing mode on/off. The component is properly integrated into the application via Canvas.tsx.

_{✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.}

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.97%. Comparing base (aaeda38) to head (3d6a326).
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #842   +/-   ##
=======================================
  Coverage   79.97%   79.97%           
=======================================
  Files         165      165           
  Lines       20108    20108           
=======================================
  Hits        16081    16081           
  Misses       4027     4027           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.