chore(Docs): Designer / 3. Reordering nodes#219
Conversation
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 22 minutes and 15 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughUpdated the "Reordering Nodes" documentation: replaced draft placeholder content with a complete guide describing three reordering methods (drag-and-drop, context-aware copy/paste variants, move-before/after), added best practices and Next Steps, and updated frontmatter metadata (date, description, removed draft flag). Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@content/docs/designer/03-reordering-nodes/_index.md`:
- Around line 50-60: The embedded <video> elements lack caption
tracks/transcripts; add accessible captions or transcripts for each video
element (e.g., the videos titled "Paste as next step", "Paste as child", "Paste
as special child" and the later videos at lines 94–106) by adding a <track
kind="captions" ...> element pointing to a .vtt file (with srclang and label)
inside each <video>, or alternatively add an adjacent transcript link/section
under each video linking to a text transcript file; ensure filenames/labels
match the existing video titles (e.g., 01-paste-as-next-step, 01-paste-as-child,
01-paste-as-special-child) so reviewers can locate and verify the new
caption/transcript assets.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 9b4d02fe-97de-4dca-a48c-8513529bb3cc
⛔ Files ignored due to path filters (6)
content/docs/designer/03-reordering-nodes/01-paste-as-child.mp4is excluded by!**/*.mp4content/docs/designer/03-reordering-nodes/01-paste-as-next-step.mp4is excluded by!**/*.mp4content/docs/designer/03-reordering-nodes/01-paste-as-special-child.mp4is excluded by!**/*.mp4content/docs/designer/03-reordering-nodes/02-move-container-nodes.mp4is excluded by!**/*.mp4content/docs/designer/03-reordering-nodes/02-move-regular-step-nodes.mp4is excluded by!**/*.mp4content/docs/designer/03-reordering-nodes/02-move-sub-containers-within-special-processors.mp4is excluded by!**/*.mp4
📒 Files selected for processing (1)
content/docs/designer/03-reordering-nodes/_index.md
| <video controls src="01-paste-as-next-step.mp4" title="Paste as next step"></video> | ||
|
|
||
| - **Paste as child** – Available only on nodes that can contain children (e.g., [`from`](https://camel.apache.org/components/latest/eips/from-eip.html), [`doTry`](https://camel.apache.org/components/latest/eips/doTry-eip.html), [`multicast`](https://camel.apache.org/components/latest/eips/multicast-eip.html)) | ||
|
|
||
|
|
||
| <video controls src="01-paste-as-child.mp4" title="Paste as child"></video> | ||
|
|
||
| - **Paste as special child** – Appears only when pasting into special processors like [`doTry`](https://camel.apache.org/components/latest/eips/doTry-eip.html) or [`choice`](https://camel.apache.org/components/latest/eips/choice-eip.html), and only if the copied content is a special node like [`when`](https://camel.apache.org/components/latest/eips/when-eip.html) or [`doCatch`](https://camel.apache.org/components/latest/eips/doCatch-eip.html) | ||
|
|
||
| 2. You can add sub-folders (ex. in manual folder) and inside add index.md | ||
|
|
||
| 3. The featured image for the main folder (ex.metadata) should be saved as featured.png | ||
| <video controls src="01-paste-as-special-child.mp4" title="Paste as special child"></video> |
There was a problem hiding this comment.
Add captions/transcripts for embedded videos.
The embedded <video> elements currently provide no caption track or linked transcript (Line 50, Line 55, Line 60, Line 94, Line 99, Line 106). Please add captions (<track kind="captions">) or adjacent transcript links for accessibility compliance.
Also applies to: 94-106
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@content/docs/designer/03-reordering-nodes/_index.md` around lines 50 - 60,
The embedded <video> elements lack caption tracks/transcripts; add accessible
captions or transcripts for each video element (e.g., the videos titled "Paste
as next step", "Paste as child", "Paste as special child" and the later videos
at lines 94–106) by adding a <track kind="captions" ...> element pointing to a
.vtt file (with srclang and label) inside each <video>, or alternatively add an
adjacent transcript link/section under each video linking to a text transcript
file; ensure filenames/labels match the existing video titles (e.g.,
01-paste-as-next-step, 01-paste-as-child, 01-paste-as-special-child) so
reviewers can locate and verify the new caption/transcript assets.
| - **[Working with Nodes](../02-nodes)**: Understand how to add and configure components in your integrations | ||
| - **[Runtime Selector](../04-runtime-selector)**: Choose the right Camel runtime for your project No newline at end of file |
There was a problem hiding this comment.
Let's comment this out since we don't have those docs yet :)
lordrip
left a comment
lordrip
left a comment
There was a problem hiding this comment.
I like the structure, maybe I would bring a bit more content from the DnD blog post here, after all, this is the documentation, so the latest information should be here
lordrip
force-pushed
the
designer_reordering_nodes
branch
2 times, most recently
from
df47ea4 to
1ddaf61
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (1)
content/docs/designer/03-reordering-nodes/_index.md (1)
48-58:⚠️ Potential issue | 🟠 MajorAdd captions or transcript links for embedded videos.
The
<video>elements still have no caption tracks/transcripts (Line 48, Line 53, Line 58, Line 92, Line 97, Line 104), which leaves this inaccessible for non-audio users.Proposed fix pattern
-<video controls src="01-paste-as-next-step.mp4" title="Paste as next step"></video> +<video controls src="01-paste-as-next-step.mp4" title="Paste as next step"> + <track kind="captions" src="01-paste-as-next-step.en.vtt" srclang="en" label="English captions" default> +</video> +<p><a href="01-paste-as-next-step.transcript.md">Transcript: Paste as next step</a></p>Apply the same pattern to each video block.
Also applies to: 92-104
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@content/docs/designer/03-reordering-nodes/_index.md` around lines 48 - 58, The three <video> elements (src="01-paste-as-next-step.mp4", "01-paste-as-child.mp4", "01-paste-as-special-child.mp4") need accessible captions/transcripts; update each <video> tag to include either a <track kind="captions" src="...vtt" srclang="en" label="English"> pointing to a WebVTT file or add a visible link to a transcript below the video (e.g., "Transcript: …") and ensure the track/transcript files are present and referenced correctly; apply the same change for the other video blocks mentioned around lines 92–104 so every embedded video has an associated caption track or transcript link.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@content/docs/designer/03-reordering-nodes/_index.md`:
- Around line 134-139: The "Next Steps" section renders empty because the
related topic list items are wrapped in HTML comments; open the "## Next Steps"
block and either uncomment the two list items (remove <!-- and --> surrounding
the two - **[Working with Nodes](../02-nodes)** and - **[Runtime
Selector](../04-runtime-selector)** lines) or replace them with active links to
the intended pages so the list displays; ensure the two link texts and targets
remain exactly as shown so navigation works.
---
Duplicate comments:
In `@content/docs/designer/03-reordering-nodes/_index.md`:
- Around line 48-58: The three <video> elements
(src="01-paste-as-next-step.mp4", "01-paste-as-child.mp4",
"01-paste-as-special-child.mp4") need accessible captions/transcripts; update
each <video> tag to include either a <track kind="captions" src="...vtt"
srclang="en" label="English"> pointing to a WebVTT file or add a visible link to
a transcript below the video (e.g., "Transcript: …") and ensure the
track/transcript files are present and referenced correctly; apply the same
change for the other video blocks mentioned around lines 92–104 so every
embedded video has an associated caption track or transcript link.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 5316aa7e-7480-4904-8e94-b47aa18d444f
📒 Files selected for processing (1)
content/docs/designer/03-reordering-nodes/_index.md
| ## Next Steps | ||
|
|
||
| 5. Use markdown features as you want and have fun! | ||
| Now that you understand node reordering, explore these related topics: | ||
|
|
||
| 6. Once finished, update the description, date and delete draft: true No newline at end of file | ||
| <!-- - **[Working with Nodes](../02-nodes)**: Understand how to add and configure components in your integrations | ||
| - **[Runtime Selector](../04-runtime-selector)**: Choose the right Camel runtime for your project --> |
There was a problem hiding this comment.
“Next Steps” currently renders empty.
Line 136 introduces related topics, but the only items are commented out (Line 138-139), so users see no next actions.
Proposed fix
## Next Steps
Now that you understand node reordering, explore these related topics:
-<!-- - **[Working with Nodes](../02-nodes)**: Understand how to add and configure components in your integrations
-- **[Runtime Selector](../04-runtime-selector)**: Choose the right Camel runtime for your project -->
+- **[Runtime Selector](../04-runtime-selector)**: Choose the right Camel runtime for your project📝 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.
| ## Next Steps | |
| 5. Use markdown features as you want and have fun! | |
| Now that you understand node reordering, explore these related topics: | |
| 6. Once finished, update the description, date and delete draft: true | |
| \ No newline at end of file | |
| <!-- - **[Working with Nodes](../02-nodes)**: Understand how to add and configure components in your integrations | |
| - **[Runtime Selector](../04-runtime-selector)**: Choose the right Camel runtime for your project --> | |
| ## Next Steps | |
| Now that you understand node reordering, explore these related topics: | |
| - **[Runtime Selector](../04-runtime-selector)**: Choose the right Camel runtime for your project |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@content/docs/designer/03-reordering-nodes/_index.md` around lines 134 - 139,
The "Next Steps" section renders empty because the related topic list items are
wrapped in HTML comments; open the "## Next Steps" block and either uncomment
the two list items (remove <!-- and --> surrounding the two - **[Working with
Nodes](../02-nodes)** and - **[Runtime Selector](../04-runtime-selector)**
lines) or replace them with active links to the intended pages so the list
displays; ensure the two link texts and targets remain exactly as shown so
navigation works.
1ddaf61 to
29dea35
Compare
lordrip
left a comment
lordrip
left a comment
There was a problem hiding this comment.
I've pushed some fixes so we can move forward to the next topic. Thanks for handling it Shivam 💪
2 participants
Summary by CodeRabbit