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

windows: show -V output #392

Merged
merged 4 commits into from
Sep 2, 2024
Merged

Conversation

vszakats
Copy link
Member

@vszakats vszakats commented Sep 2, 2024

It will be picked up from the next curl-for-win release.

Ref: curl/curl-for-win@053daa1


Screen Shot 2024-09-02 at curl-V-3

@bagder
Copy link
Member

bagder commented Sep 2, 2024

In the microsoft page I decided that the font would become too small so I made it word wrap...

@vszakats
Copy link
Member Author

vszakats commented Sep 2, 2024

In the microsoft page I decided that the font would become too small so I made it word wrap...

With the longer feature lines I found the wrapped version difficult to read:

Screen Shot 2024-09-02 at curl-V-3-nowrap

But, no strong opinion on this. Perhaps some middle ground could work? It might help to increase line spacing after the real linefeeds, but couldn't readily find a formula for it.

@bagder
Copy link
Member

bagder commented Sep 2, 2024

What do you think about this layout? With a little bit of padding after each "real" line?

Screenshot 2024-09-02 at 16-42-52 curl shipped by Microsoft

@vszakats
Copy link
Member Author

vszakats commented Sep 2, 2024

What do you think about this layout? With a little bit of padding after each "real" line?

Nice! That's what I had in mind, and it could work with curl-for-win too. How did you do it? :)

@bagder
Copy link
Member

bagder commented Sep 2, 2024

I'll make the microsoft page use this in a reusable way, then you can use the same layout. Wait a sec.

@bagder
Copy link
Member

bagder commented Sep 2, 2024

Now if you just insert #include "term.t" after the css.t include, you can make the terminal layout like this:

<div class="term">
<div class="tline">curl 8.7.1 (Windows) libcurl/8.7.1 Schannel zlib/1.3 WinIDN</div>
<div class="tline">Release-Date: 2024-03-27</div>
<div class="tline">Protocols: dict file ftp ftps http https imap imaps ipfs ipns mqtt pop3 pop3s smb smbs smtp smtps telnet tftp</div>
<div class="tline">Features: alt-svc AsynchDNS HSTS HTTPS-proxy IDN IPv6 Kerberos Largefile libz NTLM SPNEGO SSL SSPI threadsafe Unicode UnixSockets</div>
</div>

It will be picked up from the next curl-for-win release.

Used monochrome amber color on a non-pure black background.

Ref: curl/curl-for-win@053daa1
@vszakats
Copy link
Member Author

vszakats commented Sep 2, 2024

Screen Shot 2024-09-02 at curl-V-4

@vszakats vszakats merged commit 957176d into curl:master Sep 2, 2024
3 checks passed
@vszakats vszakats deleted the c4w-show-version-output branch September 2, 2024 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants