Skip to content

Tighten About pane copy and rename Links to Resources#426

Merged
FuJacob merged 1 commit into
mainfrom
about-redesign
May 30, 2026
Merged

Tighten About pane copy and rename Links to Resources#426
FuJacob merged 1 commit into
mainfrom
about-redesign

Conversation

@FuJacob

@FuJacob FuJacob commented May 30, 2026

Copy link
Copy Markdown
Owner

Summary

Trim the Support section copy in the About pane to two short sentences, relabel the Support button to "Support Cotabby" (heart icon already present), rename the "Links" section to "Resources", and shorten the Uninstall text. Brings the pane in line with the latest design mockup.

Validation

  • swiftlint lint --quiet (exit 0)
  • xcodebuild -project Cotabby.xcodeproj -scheme Cotabby -destination 'platform=macOS' build (** BUILD SUCCEEDED **)

Linked issues

None.

Risk / rollout notes

Copy and label changes only; no behavior change.

Greptile Summary

This PR tightens the copy in the About pane: the Support section blurb is condensed to two shorter sentences, the "Links" section is relabeled "Resources", the support button gains the "Cotabby" suffix, and the uninstall instructions are shortened. No logic or behavior changes are introduced.

  • The linksRow view builder retains its old name despite the section being renamed "Resources", creating a minor naming inconsistency.
  • The revised uninstall text drops the original note about manually revoking privacy permissions in System Settings, which is non-obvious macOS behavior that users attempting a full uninstall would benefit from knowing.

Confidence Score: 4/5

Safe to merge — all changes are pure UI copy with no logic, data, or behavior impact.

The change is entirely copy and label edits in a single SwiftUI view. The only notable omission is the removal of the macOS privacy-permissions reminder from the uninstall instructions, which could leave users unaware that grants like Accessibility or Input Monitoring persist after the app is deleted. The linksRow naming mismatch after the "Resources" rename is a minor readability concern for future contributors.

Cotabby/UI/Settings/Panes/AboutPaneView.swift — specifically the uninstall text block and the linksRow naming.

Important Files Changed

Filename Overview
Cotabby/UI/Settings/Panes/AboutPaneView.swift Copy-only changes: trims support section text, renames "Links" section to "Resources", relabels support button, and shortens uninstall instructions — the private view builder linksRow retains its old name after the section rename, and the uninstall text drops a user-facing note about manually revoking privacy permissions in System Settings.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[AboutPaneView] --> B[Section: About Header]
    A --> C[Section: Support]
    A --> D["Section: Resources (was: Links)"]
    A --> E[Section: Uninstall]

    C --> C1["Label: trimmed 2-sentence copy"]
    C --> C2["Button: 'Support Cotabby' ❤️ (was: 'Support')"]

    D --> D1[GitHub Repository link]
    D --> D2[Wiki & Contributor Guide link]
    D --> D3[Acknowledgements button]

    E --> E1["'Remove Cotabby from Applications…' (shortened)"]
    E --> E2["⚠️ Privacy permissions note removed"]
Loading

Fix All in Codex Fix All in Claude Code

Reviews (1): Last reviewed commit: "Tighten About pane copy and rename Links..." | Re-trigger Greptile

Greptile also left 2 inline comments on this PR.

@FuJacob FuJacob merged commit 7380f93 into main May 30, 2026
4 checks passed
Section { aboutHeader }
Section("Support") { supportRow }
Section("Links") { linksRow }
Section("Resources") { linksRow }

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 The section label was renamed to "Resources" but the private view builder is still called linksRow. This is a minor naming inconsistency that could cause confusion for future contributors.

Suggested change
Section("Resources") { linksRow }
Section("Resources") { resourcesRow }

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Fix in Codex Fix in Claude Code

Comment on lines 107 to 110
Text(
"Drag Cotabby.app from Applications to the Trash. "
+ "To remove leftover data, also delete ~/Library/Application Support/Cotabby. "
+ "Privacy permissions can only be revoked in System Settings → Privacy & Security."
"Remove Cotabby from Applications. To fully clean up app data, "
+ "delete ~/Library/Application Support/Cotabby."
)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 Dropped macOS privacy-permissions guidance

The original text included "Privacy permissions can only be revoked in System Settings → Privacy & Security." On macOS, dragging an app to the Trash does not automatically revoke permissions like Accessibility, Input Monitoring, or Full Disk Access. Without this note, users who want a clean uninstall won't know those grants persist until they find them manually. Consider keeping at least a short mention, e.g. "Privacy permissions granted in System Settings are not removed automatically."

Fix in Codex Fix in Claude Code

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.

1 participant