-
Notifications
You must be signed in to change notification settings - Fork 26.7k
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 the usage text more visible #3693
Conversation
Some ideas to make it even more obvious:
Dunno, just some ideas. LGTM either way. If you do go the ANSI escape route, make sure it's disabled when you have a dumb terminal detected? We have code to do that already. |
Perhaps a welcome message the first time the tool is run, with a prominent message about analytics, and a 2 second delay before we continue with running the first command? Red text to me would signify an error condition; this is more something that we want to emphasize. I think the delay will help the user notice the message, instead of having it scroll quickly to the top of the screen. |
It could be green text, or purple. :-) I'm worried about a delay because it'll make us look slow. You could always try |
|
Well that looks awesome. |
printStatus(''); | ||
printStatus(''' | ||
╔════════════════════════════════════════════════════════════════════════════════════════════════════╗ | ||
║ Welcome to Flutter! - Flutter version $versionString - https://flutter.io ║ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
getVersionString can return variable width strings, we should probably figure out the spacing here on the fly rather than hard-coding it. Feel free to file a bug on me if you want me to take care of it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh yeah; I was thinking that it was hard-coded to a specific length, but that's just for the first half of the version, not the branch name part.
Use whitespace and indent to make the first-run usage text more visible:
I experimented with changing where it displayed but had some issues with long and short running commands; happy to iterate on the look.
@Hixie