Skip to content

Improve screenshot screenrecord documentation#5399

Open
marcmav wants to merge 3 commits intobasecamp:devfrom
marcmav:improve-screenshot-screenrecord-documentation
Open

Improve screenshot screenrecord documentation#5399
marcmav wants to merge 3 commits intobasecamp:devfrom
marcmav:improve-screenshot-screenrecord-documentation

Conversation

@marcmav
Copy link
Copy Markdown

@marcmav marcmav commented Apr 23, 2026

improved documentation for easier accessing when changing defaults, added absolute paths so it can be easy accessible

Copilot AI review requested due to automatic review settings April 23, 2026 11:55
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates Omarchy’s screenshot/screenrecord documentation comments to make it easier for users to discover where to change default save locations and related environment variables.

Changes:

  • Clarify where to set screenshot directory overrides (OMARCHY_SCREENSHOT_DIR / XDG_PICTURES_DIR).
  • Clarify where to set screenrecording directory overrides (OMARCHY_SCREENRECORD_DIR / XDG_VIDEOS_DIR).
  • Adjust the uwsm default editor comment.

Reviewed changes

Copilot reviewed 1 out of 3 changed files in this pull request and generated 1 comment.

File Description
config/uwsm/default Tweaks comment describing the default editor and how to use VS Code.
bin/omarchy-cmd-screenshot Expands top-of-file comment to include paths for configuring screenshot output dir.
bin/omarchy-cmd-screenrecord Expands top-of-file comment to include paths for configuring screenrecord output dir.

Tip

If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread config/uwsm/default
export TERMINAL=xdg-terminal-exec

# Use code for VSCode
# Default editor is Neovim, use <code> for VSCode
Copy link

Copilot AI Apr 23, 2026

Choose a reason for hiding this comment

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

This comment is less clear than the previous # Use code for VSCode. In a shell config file, <code> looks like a placeholder/HTML. Consider wording like “To use VSCode, set EDITOR=code” so it’s unambiguous what value should be used.

Suggested change
# Default editor is Neovim, use <code> for VSCode
# Default editor is Neovim; to use VSCode, set EDITOR=code

Copilot uses AI. Check for mistakes.
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