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

Add a switch for exporting settings #19

Merged
merged 2 commits into from
Apr 25, 2018
Merged

Conversation

zadjii-msft
Copy link
Member

As suggested by #6. colortool -o <filename> will write the current properties to <filename>, in ini file format.

@@ -208,6 +214,37 @@ static bool SetDefaults(uint[] colorTable)
return true;
}

static bool ExportCurrentAsIni(string outputPath)
Copy link
Member

Choose a reason for hiding this comment

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

Since there are multiple ways we may want to export, it would be nice to have some interface we could program against (ie ISchemeExporter in contrast to ISchemeParser) so ini, iterm, etc could be used

Copy link

Choose a reason for hiding this comment

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

Indeed, exporting to .itermcolors should be possible, not only to .ini

Copy link
Member

Choose a reason for hiding this comment

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

So as to not back up this code review any longer, I'm going to "Won't Fix" this and link it to the new issue #160.

My goal at this moment is to clear this backlog of pulls. Thanks.

@microsoft microsoft deleted a comment from msftclas Sep 27, 2017
}
else
{
Console.WriteLine("Failed to get conosle information.");
Copy link

@oscarsiauw oscarsiauw Nov 21, 2017

Choose a reason for hiding this comment

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

Typo "conosle" should be "console"

Copy link
Member

Choose a reason for hiding this comment

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

Fixed.

@miniksa miniksa self-assigned this Apr 25, 2018
@miniksa miniksa added the Product-Colortool This is a problem with the utility application ColorTool for manipulating the console color palette. label Apr 25, 2018
@miniksa miniksa merged commit d66df17 into master Apr 25, 2018
@miniksa miniksa deleted the dev/zadjii/f/output-current branch April 25, 2018 21:18
carlos-zamora pushed a commit that referenced this pull request Aug 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Product-Colortool This is a problem with the utility application ColorTool for manipulating the console color palette.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants