Skip to content

Add option to use theme provided cell background (#675) #679

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.

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
merged 3 commits into from
Mar 11, 2025

Conversation

kylebutts
Copy link
Contributor

Updates configuration for cell background color. See #677 for discussion.

Updates:
quarto.cells.background.enabled -> quarto.cells.background.color which now has three options: "default", "off", and "useTheme". useTheme will use the notebook.selectedCellBackground color from the VS Code theme.

Renamed default color options:
quarto.cells.background.light -> quarto.cells.background.lightDefault and quarto.cells.background.dark -> quarto.cells.background.darkDefault

Copy link
Collaborator

@juliasilge juliasilge left a comment

Choose a reason for hiding this comment

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

Thank you again so much for this contribution and your willingness to iterate on it, @kylebutts! ❤️

This is working great for me and looks good in the settings UI:

Screenshot 2025-03-11 at 10 58 20 AM

Screenshot 2025-03-11 at 10 58 57 AM

I also added a deprecation warning for the old setting, to hopefully help any folks who did set it away from the default:

Screenshot 2025-03-11 at 1 06 38 PM

@juliasilge juliasilge merged commit fc84895 into quarto-dev:main Mar 11, 2025
1 check passed
@juliasilge
Copy link
Collaborator

@kylebutts we'll do a release fairly soon for the Quarto extension, but if you want to use this in the meantime, I think the easiest thing to do is get the artifact from this GH action run, unzip it, and then "Install from VSIX" in VS Code or Positron:

https://github.com/quarto-dev/quarto/actions/runs/13796690796

@juliasilge
Copy link
Collaborator

Fixing the deprecated setting name in #680 🙈

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.

2 participants