Skip to content

Conversation

@iparaskev
Copy link
Contributor

@iparaskev iparaskev commented Feb 2, 2026

Summary by CodeRabbit

  • Documentation
    • Renamed the drawing guide subsection to emphasize "How to use drawing as a Controller."
    • Replaced the old Limitations section with a new "How to use drawing as a Sharer" workflow.
    • Added step-by-step Sharer instructions, a Sharer workflow video, and guidance on modes and exit behavior.
    • Kept existing Controller content and video.
    • Updated features overview to include Drawing in the features list.

@netlify
Copy link

netlify bot commented Feb 2, 2026

Deploy Preview for hoppdocs ready!

Name Link
🔨 Latest commit 5904429
🔍 Latest deploy log https://app.netlify.com/projects/hoppdocs/deploys/6988bbc3b3227b0008cac4d3
😎 Deploy Preview https://deploy-preview-251--hoppdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link

coderabbitai bot commented Feb 2, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Documentation updated to add a Sharer workflow for the drawing feature, rename the Controller section, embed a new Sharer video, and update the features index to link the drawing page.

Changes

Cohort / File(s) Summary
Drawing docs
docs/src/content/docs/features/drawing.mdx
Restructured drawing doc: renamed Controller section, replaced "Limitations" with a new "How to use drawing as a Sharer" guide, added step-by-step Sharer instructions and a new embedded Sharer video; preserved Controller content and existing video.
Features index
docs/src/content/docs/index.mdx
Updated Features card text and link list to include "Drawing" and point to /features/drawing/.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I hopped in with a pen and a grin,
Sharers now sketch where frames begin,
Controllers kept the easel fine,
Now both can doodle, mark, and sign —
A little hop, a lively line! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'add: sharer drawing docs' clearly and concisely summarizes the main change: adding documentation for drawing functionality in the Sharer role.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update_drawing_docs

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

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

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@docs/src/content/docs/features/drawing.mdx`:
- Around line 31-35: The option label text "Persist until right click" should be
hyphenated for consistency; update the docs text in the drawing mode options
list (the list item that currently reads "Persist until right click") to use
"Persist until right-click" so the label is standardly hyphenated.

Comment on lines +31 to +35
1. Click the drawing button in the call controls to enable drawing mode.
2. You can configure the drawing mode from the three dots menu:
- **Temporary**: Drawings will be cleared after 3 seconds (default).
- **Persist until right click**: Drawings remain on screen until you right-click to clear them.
3. To exit drawing mode, press **Escape**.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Hyphenate “right-click” in the option label.
Minor doc polish to match standard hyphenation.

🔤 Proposed edit
-   - **Persist until right click**: Drawings remain on screen until you right-click to clear them.
+   - **Persist until right-click**: Drawings remain on screen until you right-click to clear them.
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
1. Click the drawing button in the call controls to enable drawing mode.
2. You can configure the drawing mode from the three dots menu:
- **Temporary**: Drawings will be cleared after 3 seconds (default).
- **Persist until right click**: Drawings remain on screen until you right-click to clear them.
3. To exit drawing mode, press **Escape**.
1. Click the drawing button in the call controls to enable drawing mode.
2. You can configure the drawing mode from the three dots menu:
- **Temporary**: Drawings will be cleared after 3 seconds (default).
- **Persist until right-click**: Drawings remain on screen until you right-click to clear them.
3. To exit drawing mode, press **Escape**.
🧰 Tools
🪛 LanguageTool

[grammar] ~34-~34: Use a hyphen to join words.
Context: ...ds (default). - Persist until right click: Drawings remain on screen until...

(QB_NEW_EN_HYPHEN)

🤖 Prompt for AI Agents
In `@docs/src/content/docs/features/drawing.mdx` around lines 31 - 35, The option
label text "Persist until right click" should be hyphenated for consistency;
update the docs text in the drawing mode options list (the list item that
currently reads "Persist until right click") to use "Persist until right-click"
so the label is standardly hyphenated.

@konsalex konsalex merged commit 4fbc3fb into main Feb 8, 2026
6 of 8 checks passed
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.

2 participants