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

Options architecture #51

Open
zspitz opened this issue Sep 28, 2020 · 0 comments
Open

Options architecture #51

zspitz opened this issue Sep 28, 2020 · 0 comments
Labels
options Passing additional configuration options to ToString
Milestone

Comments

@zspitz
Copy link
Owner

zspitz commented Sep 28, 2020

Sometimes a given rendered will expose various options, e.g. text wrapping in the code renderers, or the using / Imports / static usings when rendering a type name, or the number of spaces to indent.

If a renderer supports various options, there should be a class describing those options, and a static instance of the class e.g. at CSharpRenderOptions.Current. The options should not be set per call, but rather per visitor/writer.

This also allows the renderer at the C# key to be overridden with another rendering function, perhaps with its own option mechanism.

As long as these option classes are serializable, the visualizer can support passing them back and forth.

We'll open individual issues for each writer/visitor, to describe the available options.

@zspitz zspitz added this to the 3.3 milestone Oct 1, 2020
@zspitz zspitz modified the milestones: 3.5, 4.0 Jun 27, 2021
@zspitz zspitz added the options Passing additional configuration options to ToString label Jul 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
options Passing additional configuration options to ToString
Projects
None yet
Development

No branches or pull requests

1 participant