docs(gui): add screenshots for all seven web UI screens#2061
Open
Om-Rohilla wants to merge 1 commit into
Open
Conversation
Adds webui-tasks, webui-agents, webui-approvals, webui-audit, webui-costs, webui-fleet, and webui-settings screenshots to docs/assets/. Updates docs/gui/screens.md to embed them in the existing Screenshots section, and adds a representative image to the Surface section in docs/web-ui.md. Contributes to sipyourdrink-ltd#1262.
Reviewer's guide (collapsed on small PRs)Reviewer's GuideAdds new web UI screenshots to the docs assets and embeds them into the GUI documentation pages, wiring each of the seven top-level screens into the existing Screenshots and Surface sections. File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds UI screenshots to the documentation to better illustrate the Web UI surfaces and available routes.
Changes:
- Embed a Tasks view screenshot into
docs/web-ui.md. - Add screenshots for multiple route screens into
docs/gui/screens.md.
Reviewed changes
Copilot reviewed 2 out of 9 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| docs/web-ui.md | Adds an inline screenshot for the Tasks view in the Web UI docs. |
| docs/gui/screens.md | Adds screenshots for additional UI routes to the GUI screens reference. |
Comments suppressed due to low confidence (1)
docs/gui/screens.md:1
- The text says 'Five routes' but the newly added screenshots immediately below indicate more than five (Tasks, Agents, Approvals, Audit, Costs, Fleet, Settings). Please update the route count to match reality (or rephrase to avoid a hard-coded number) so the documentation doesn’t become misleading.
---
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (7)
📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR adds screenshot references to the Bernstein documentation. The GUI screens documentation was expanded with route-specific screenshot links for Tasks, Agents, Approvals, Audit, Costs, Fleet, and Settings. The web UI documentation now includes a Tasks view screenshot link to visually illustrate the web UI interface. ChangesDocumentation Screenshots
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 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 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 |
Author
|
Hi, I've submitted PR #2061 covering the README screenshots sub-task. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this does
Adds screenshots for all seven top-level web UI screens (Tasks, Agents, Approvals, Audit, Costs, Fleet, Settings) to
docs/assets/.Embeds them in
docs/gui/screens.mdunder the existing## Screenshotssection, and adds a representative Tasks screenshot to the## Surfacesection ofdocs/web-ui.md.Contributes to #1262 (README screenshots / GIFs sub-task from the Web UI contributor welcome mat).
Files changed
docs/assets/webui-tasks.pngdocs/assets/webui-agents.pngdocs/assets/webui-approvals.pngdocs/assets/webui-audit.pngdocs/assets/webui-costs.pngdocs/assets/webui-fleet.pngdocs/assets/webui-settings.pngdocs/gui/screens.md## Screenshotssectiondocs/web-ui.md## SurfacesectionHow screenshots were taken
BERNSTEIN_AUTH_DISABLED=1 bernstein gui serveathttp://127.0.0.1:8052/ui/webui-*.pngconvention already established indocs/assets/Checklist
webui-*.pngindocs/assets/)docs/gui/screens.mdupdated (screenshots embedded in existing section)docs/web-ui.mdupdated (representative image added to Surface overview)Summary by Sourcery
Add embedded screenshots of all top-level web UI screens to the GUI documentation.
Documentation:
Summary by CodeRabbit