Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

Update MainService.Plugins.cs #713

Merged
merged 10 commits into from
Jan 14, 2021
Merged

Conversation

chenzhitong
Copy link
Member

@erikzhang
Copy link
Member

I don't think it can align the texts.

@chenzhitong chenzhitong reopened this Jan 14, 2021
@chenzhitong
Copy link
Member Author

before
image

after
image

@chenzhitong
Copy link
Member Author

I don't think it can align the texts.

Now it can.

@@ -107,7 +107,7 @@ private void OnPluginsCommand()
foreach (Plugin plugin in Plugin.Plugins)
{
if (plugin is Logger) continue;
Console.WriteLine("\t" + plugin.Name + "\t" + plugin.Description);
Console.WriteLine(string.Format("\t{0,-10}\t{1}", plugin.Name, plugin.Description));
Copy link
Member

Choose a reason for hiding this comment

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

ApplicationLogs has 15 characters.

Copy link
Member Author

@chenzhitong chenzhitong Jan 14, 2021

Choose a reason for hiding this comment

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

image
The Tab field is 8 characters in size

Copy link
Member

Choose a reason for hiding this comment

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

Maybe {0,-15} is better?

Copy link
Member Author

Choose a reason for hiding this comment

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

{0,-15}
image

{0,-16}
image

Copy link
Member

Choose a reason for hiding this comment

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

15 is better.

Copy link
Member Author

@chenzhitong chenzhitong Jan 14, 2021

Choose a reason for hiding this comment

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

I removed \t and set Name.MaxLength = 20

image

@erikzhang
Copy link
Member

Why revert?

@chenzhitong

This comment has been minimized.

@erikzhang erikzhang merged commit 28ddf76 into neo-project:master Jan 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants