Skip to content
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

Make Notebook preferences registration substitutable #13926

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

cdamus
Copy link
Contributor

@cdamus cdamus commented Jul 15, 2024

What it does

Implement the commonly employed pattern for preference contribution registration to enable substitution of the Notebook preference schema.

Fixes #13913

How to test

Verify that the Notebook preferences appear in the Settings panel as before.
Open some kind of notebook and verify that the specific preference settings have the expected effect on notebook behaviour, output, etc. For example, in a GitHub Issues Notebook, create a code cell, enter some code, and toggle the line numbers preference setting to see how line numbers appear or not in the cell editor.

Follow-ups

No follow-up is identified at this time.

Review checklist

Reminder for reviewers

Implement the commonly employed pattern for preference contribution
registration to enable substitution of the Notebook preference schema.

Fixes eclipse-theia#13913

Signed-off-by: Christian W. Damus <cdamus.ext@eclipsesource.com>
Copy link
Contributor

@jonah-iden jonah-iden left a comment

Choose a reason for hiding this comment

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

Seems to all work fine and code looks good

@msujew msujew added preferences issues related to preferences notebook issues related to notebooks labels Jul 15, 2024
@msujew msujew merged commit 7f928f2 into eclipse-theia:master Jul 16, 2024
14 checks passed
@cdamus cdamus deleted the issues/13913 branch July 16, 2024 12:43
@sgraband sgraband added this to the 1.52.0 milestone Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notebook issues related to notebooks preferences issues related to preferences
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Align Notebook preferences registration with other preferences
4 participants