Skip to content

Clarify screenshot guidance for contributions #15

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
alvst wants to merge 1 commit into
base: latest-preview-style
Choose a base branch
from
Draft

Clarify screenshot guidance for contributions #15

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions CONTRIBUTING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,13 +37,13 @@ open packaging/Pesty.app
- Keep it dependency-free. Prefer system frameworks (AppKit, SwiftUI, Carbon, ServiceManagement).
- Match the existing style. Small, focused changes; no unrelated refactors in the same PR.
- Test on both Apple Silicon and Intel where it matters (the release is universal).
- UI changes: include a before/after screenshot of the strip.
- UI changes: include before/after screenshots when they help explain a meaningful visual change; do not add them for every small UI tweak.

## Pull requests

1. Fork and branch from `main`.
2. Make your change; ensure `swift build` is clean (no warnings).
3. Open a PR with a clear description and screenshots for UI changes.
3. Open a PR with a clear description and screenshots when a UI change benefits from them.

## Good first issues

Expand Down
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ Pesty reimplements the parts of Paste people use every day - the slide-up strip,

## Contributing

PRs welcome - see [CONTRIBUTING.md](CONTRIBUTING.md). Good first issues: large preview pane, drag-and-drop out of cards, strip resize handle, iOS/iPad companion, more content-type renderers.
PRs welcome - see [CONTRIBUTING.md](CONTRIBUTING.md). Good first issues: large preview pane, drag-and-drop out of cards, strip resize handle, iOS/iPad companion, more content-type renderers. Add screenshots when the UI changes in a way that screenshots help explain, rather than for every small change.

## License

Expand Down