Skip to content

Stack About > Support button below the copy#427

Merged
FuJacob merged 1 commit into
mainfrom
fix/about-support-button-layout
May 30, 2026
Merged

Stack About > Support button below the copy#427
FuJacob merged 1 commit into
mainfrom
fix/about-support-button-layout

Conversation

@FuJacob

@FuJacob FuJacob commented May 30, 2026

Copy link
Copy Markdown
Owner

Summary

The Ko-fi button on the About pane's Support section sat in LabeledContent's value column, placing it to the right of a two-paragraph mission statement. The wall of text visually competed with a small action and read awkwardly. Move the section to a plain VStack so the copy reads top-down and the call-to-action lands underneath it.

Validation

xcodebuild -project Cotabby.xcodeproj -scheme Cotabby \
  -destination 'platform=macOS' build
# ** BUILD SUCCEEDED **

Visual layout change only; verified locally by reading the diff against the screenshot the user attached.

Linked issues

None.

Risk / rollout notes

Layout only; no behavior change.

Greptile Summary

Restructures the Support section of the About pane by replacing LabeledContent with a VStack, so the Ko-fi button renders below the two-paragraph mission statement rather than alongside it in the value column. No logic or behavioral changes are introduced.

  • Removes LabeledContent wrapper in supportRow and replaces it with VStack(alignment: .leading, spacing: 12), grouping the text copy and the Link button vertically.
  • The URL guard (if let supportURL = URL(string:)) and button styling are preserved unchanged from the original implementation.

Confidence Score: 5/5

Layout-only change with no behavioral, logic, or data-handling modifications — safe to merge.

The diff moves a single SwiftUI button from the value column of a LabeledContent into a VStack below the text copy. All URL construction, button styling, and section structure are preserved intact. There is nothing here that could affect runtime behavior.

No files require special attention.

Important Files Changed

Filename Overview
Cotabby/UI/Settings/Panes/AboutPaneView.swift Replaced LabeledContent with a VStack in supportRow, moving the Ko-fi button below the mission-statement copy instead of beside it. Pure layout refactor — no logic, data, or behavior changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    subgraph Before["Before (LabeledContent)"]
        LC[LabeledContent]
        LC -->|label column| TXT["VStack — mission text\n(two paragraphs)"]
        LC -->|value column| BTN["Link — Support Cotabby\n(borderedProminent button)"]
    end

    subgraph After["After (VStack)"]
        VS[VStack spacing=12]
        VS --> TXT2["VStack spacing=8 — mission text\n(two paragraphs)"]
        VS --> BTN2["Link — Support Cotabby\n(borderedProminent button)"]
    end

    Before -.->|"layout refactor"| After
Loading

Reviews (1): Last reviewed commit: "Stack About > Support button below the c..." | Re-trigger Greptile

The Ko-fi button sat in LabeledContent's value column, putting it to the right
of a two-paragraph mission statement. That made the wall of text visually
compete with a small action and was awkward to scan. Move to a simple VStack
so the paragraphs read top-down and the call-to-action lands underneath them.
@FuJacob FuJacob merged commit f446922 into main May 30, 2026
4 checks passed
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