Skip to content

CLI CATANATON Banner#307

Open
bcollazo wants to merge 2 commits intomasterfrom
cli-banner
Open

CLI CATANATON Banner#307
bcollazo wants to merge 2 commits intomasterfrom
cli-banner

Conversation

@bcollazo
Copy link
Copy Markdown
Owner

@bcollazo bcollazo commented May 7, 2025

Making the entrypoint look like:

Screenshot 2025-05-06 at 11 38 54 PM

👀 👀

@zarns @tonypr @pachewise what do you guys think? Is it too "hacky" 😬 :)

@netlify
Copy link
Copy Markdown

netlify bot commented May 7, 2025

Deploy Preview for catanatron-staging canceled.

Name Link
🔨 Latest commit 316ee37
🔍 Latest deploy log https://app.netlify.com/sites/catanatron-staging/deploys/681ad6fd4557670008e0a127

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 14874660892

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.07%) to 95.981%

Files with Coverage Reduction New Missed Lines %
catanatron_core/catanatron/state_functions.py 1 96.69%
Totals Coverage Status
Change from base Build 14874508140: -0.07%
Covered Lines: 1385
Relevant Lines: 1443

💛 - Coveralls

Copy link
Copy Markdown
Contributor

@pachewise pachewise May 7, 2025

Choose a reason for hiding this comment

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

Thematically, I think it's good. Stylistically - maybe make it smaller, and add a border to it? (Also, curious about the blue color?)

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.

Agreed 💯 I'd say it's a little too big. I asked claude to generate a color scheme and the secondary accent color matches almost exactly 🤣 It's always been hard to work around the traditional color scheme and make it feel cyberpunk-y because the player colors and resource colors clash no matter how what.

image

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.

4 participants