feat: docs panel, prompts icons, sessions icons, v0.0.52#200
Conversation
- Change default registry preset from official to npmmirror - Remove official registry button from docs panel - Add max-width to command display to keep copy button visible - Update status strip fallback to show npmmirror
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe default npm install registry preset is changed from ChangesRegistry Default, Docs Layout, and Agents Editor Updates
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ast-grep (0.43.0)web-ui/res/web-ui-render.precompiled.jsThanks 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 |
- Position copy button absolutely to stay visible during scroll - Add right padding to command box for button space - Update mobile responsive layout for absolute positioning
…ility Switch command display from horizontal scroll to word-break wrapping, change copy button from absolute positioning to flex layout so it remains visible regardless of command length.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (1)
web-ui/styles/docs-panel.css (1)
122-124: ⚡ Quick winUse less aggressive command wrapping to preserve token readability.
Line 123 uses
word-break: break-all, which can split command tokens (--flags, package names) mid-token and make copy/scan verification harder. Prefer natural token-preserving wrapping.Suggested CSS adjustment
.docs-command-box .install-command { flex: 1; min-width: 0; @@ - white-space: normal; - word-break: break-all; - overflow-wrap: break-word; + white-space: normal; + word-break: normal; + overflow-wrap: anywhere;🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@web-ui/styles/docs-panel.css` around lines 122 - 124, The word-break property in the docs-panel.css file is set to break-all, which aggressively breaks command tokens in the middle and reduces readability. Replace the word-break: break-all property with word-break: break-word or simply remove it entirely and rely on the overflow-wrap: break-word property that is already present, which will preserve token boundaries while still allowing wrapping when necessary.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@web-ui/styles/docs-panel.css`:
- Around line 122-124: The word-break property in the docs-panel.css file is set
to break-all, which aggressively breaks command tokens in the middle and reduces
readability. Replace the word-break: break-all property with word-break:
break-word or simply remove it entirely and rely on the overflow-wrap:
break-word property that is already present, which will preserve token
boundaries while still allowing wrapping when necessary.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 8fae1e56-159f-4a36-8504-583cfcfb3e3f
📒 Files selected for processing (1)
web-ui/styles/docs-panel.css
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: ci (18)
Replace all 6 text buttons in the prompts editor toolbar with inline SVG icons and title tooltips. Export function now generates .md files with dynamic prefix (agents/claude) based on active sub-tab.
Replace clipboard icon with chain-link icon for copy-link button and folder icon for copy-path button to distinguish the two actions.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@web-ui/res/web-ui-render.precompiled.js`:
- Around line 6025-6156: The icon-only buttons in the toolbar
(exportAgentsContent, copyAgentsContent, pasteAgentsContent, loadPromptsContent,
resetAgentsDiffState, and applyAgentsContent) are missing accessible names that
screen readers can announce. Currently they only have a title attribute which is
not accessible to assistive technology. Add an aria-label attribute to each
button element in the props object, using the same translation key that is used
for the title attribute, so that screen readers can properly announce the
button's purpose alongside the visual tooltip.
🪄 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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 9df12bb3-b6d8-48d4-97e8-061e925774f2
📒 Files selected for processing (3)
web-ui/modules/app.methods.session-actions.mjsweb-ui/partials/index/panel-prompts.htmlweb-ui/res/web-ui-render.precompiled.js
📜 Review details
🔇 Additional comments (3)
web-ui/partials/index/panel-prompts.html (2)
45-65: LGTM!
68-82: LGTM!web-ui/modules/app.methods.session-actions.mjs (1)
343-354: LGTM!
ymkiux
changed the title
|
Caution Failed to replace (edit) comment. This is likely due to insufficient permissions or the comment being deleted. Error details |
1 participant
Summary
Summary by CodeRabbit