Make pulling output use primary colors

[gtsiolis]

See DES-152 for more context.

BEFORE	AFTER

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: dd8d9992-5854-458b-b72d-33955dc229e0

📥 Commits

Reviewing files that changed from the base of the PR and between cd3f6b3 and 39c7991.

📒 Files selected for processing (2)

• internal/ui/components/pull_progress.go
• internal/ui/components/spinner.go

---

📝 Walkthrough

Walkthrough

The pull request modifies two UI component rendering methods in the spinner and progress components. One change adds styling to a progress label, while the other removes styling from spinner text. Both affect how text is rendered during component display.

Changes

Cohort / File(s)	Summary
UI Component Styling Updates internal/ui/components/pull_progress.go, internal/ui/components/spinner.go	Modified View() methods to adjust text styling: pull_progress now applies Secondary style to phase labels, while spinner removes Secondary style from appended text.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Collapse image pull progress into loading indicator #82: Modifies the same UI component methods (PullProgress.View and Spinner.View) with changes to how labels and text are styled and rendered.

Suggested reviewers

• carole-lavillonniere

🚥 Pre-merge checks | ✅ 2

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Make pulling output use primary colors' is directly related to the changes. The code modifications involve altering color styling in pull progress and spinner components.
Description check	✅ Passed	The description includes before/after screenshots and references DES-152, demonstrating visual changes to the pulling output. It is clearly related to the changeset.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch george/make-pulling-primary-output

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[silv-io]

Yep, makes way more sense like this!

This will change all other occurrences of the spinner, though (login, logout, etc.)

[silv-io]

Maybe we want to use the Secondary style for the percentage as well? We can do that in the bubbles.progress styles when creating it. WDYT?