Skip to content

default#19

Merged
botterYosuke merged 1 commit intomainfrom
sasa/mode
Jan 18, 2026
Merged

default#19
botterYosuke merged 1 commit intomainfrom
sasa/mode

Conversation

@botterYosuke
Copy link
Owner

@botterYosuke botterYosuke commented Jan 18, 2026

📝 Summary

Closes #

🔍 Description of Changes

📋 Checklist

  • I have read the contributor guidelines.
  • For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
  • Tests have been added for the changes made.
  • Documentation has been updated where applicable, including docstrings for API changes.
  • Pull request title is a good summary of the changes - it will be used in the release notes.

Summary by CodeRabbit

  • New Features
    • Introduced "grid" as a new width layout option for display configuration
    • Changed default width layout from "medium" to "grid"

✏️ Tip: You can customize this high-level summary in your review settings.

@vercel
Copy link

vercel bot commented Jan 18, 2026

👋 It looks like the test check failed.

To fix this locally, please run:

pnpm turbo test

@botterYosuke botterYosuke merged commit 9172576 into main Jan 18, 2026
7 of 12 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Jan 18, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The changes introduce "grid" as a new width configuration option and establish it as the default display width, updating both frontend config schema and backend configuration from "medium" to "grid" across multiple prefault declarations.

Changes

Cohort / File(s) Change Summary
Width Configuration Default Update
frontend/src/core/config/config-schema.ts, marimo/_config/config.py
Adds "grid" to WidthType Literal in backend; updates AppConfigSchema width field prefault and top-level default object in frontend schema from "medium" to "grid"; aligns UserConfigSchema display.default_width prefault value correspondingly; synchronizes DEFAULT_CONFIG DisplayConfig in backend

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 wiggles nose
From "medium" to "grid" the defaults now gleam,
Frontend and backend aligned like a dream,
Width options expand with options so fine,
The grid takes its throne—a config design! ✨

✨ Finishing touches
  • 📝 Generate docstrings

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

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

@github-actions
Copy link

👋 It looks like the dedupe check failed.

To fix this locally, please run:

pnpm dedupe

@botterYosuke botterYosuke deleted the sasa/mode branch January 18, 2026 12:10
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