Skip to content

Tighten About pane copy and rename Links to Resources #426

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

Merged
FuJacob merged 1 commit into from
May 30, 2026
+8 −12
Merged

Tighten About pane copy and rename Links to Resources #426

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
20 changes: 8 additions & 12 deletions Cotabby/UI/Settings/Panes/AboutPaneView.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ struct AboutPaneView: View {
SettingsPaneScaffold {
Section { aboutHeader }
Section("Support") { supportRow }
Section("Links") { linksRow }
Section("Resources") { linksRow }

@greptile-apps greptile-apps Bot May 30, 2026

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

Section("Uninstall") { uninstallText }
}
.sheet(isPresented: $isShowingAcknowledgements) {
Expand Down Expand Up @@ -58,24 +58,21 @@ struct AboutPaneView: View {
LabeledContent {
if let supportURL = URL(string: "https://ko-fi.com/cotabby") {
Link(destination: supportURL) {
Label("Support", systemImage: "heart.fill")
Label("Support Cotabby", systemImage: "heart.fill")
}
.buttonStyle(.borderedProminent)
.tint(.blue)
}
} label: {
VStack(alignment: .leading, spacing: 8) {
Text(
"Cotabby started from a simple belief: AI should run on your device, respect your privacy, "
+ "and remain open to everyone. That's why Cotabby is open source, local-first, and built "
+ "to give users full control over their own hardware and data."
"Cotabby started from a simple belief: AI should run on your device, "
+ "respect your privacy, and remain open to everyone."
)

Text(
"We're building Cotabby in our spare time, one release at a time. Every feature, bug fix, "
+ "and support reply takes real time and care from two university students trying to keep "
+ "this project alive. If Cotabby has been useful to you, please consider supporting our "
+ "mission."
"We're building Cotabby in our spare time, one release at a time. "
+ "If Cotabby has helped you, your support helps us keep improving it."
)
}
.foregroundStyle(.secondary)
Expand Down Expand Up @@ -108,9 +105,8 @@ struct AboutPaneView: View {
@ViewBuilder
private var uninstallText: some View {
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."
)
Comment on lines 107 to 110

@greptile-apps greptile-apps Bot May 30, 2026

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

.font(.caption)
.foregroundStyle(.secondary)
Expand Down