search

Julien-cpsn / ATAC

A simple API client (postman like) in your terminal
https://atac.julien-cpsn.com/
MIT License
1.86k stars 81 forks source link

Add headers highlighting #31

Closed savchenko closed 4 months ago

savchenko commented 5 months ago

Response body has syntax highlighting. Can something similar be applied to the headers section? At least to differentiate a header from its content.

Julien-cpsn commented 5 months ago

Hello, I am note quite sure I understand. You mean for the response headers?

Julien-cpsn commented 5 months ago

@savchenko

savchenko commented 5 months ago

The "headers" section on the right. Request headers are split in two columns, while response headers are displayed as simple text with no columns or highlighting:

image

Is it possible to make them more readable?

Julien-cpsn commented 5 months ago

yes sure I'll highlight them more. It will not be displayed as a table like on the request part though

PS: Don't forget to update the app :))

savchenko commented 5 months ago

yes sure I'll highlight them more. It will not be displayed as a table like on the request part though

Any reason not to? Depending on the length of the longest header it might be not the best utilisation of the free space, however in situation with many similar headers is might be handy.

Julien-cpsn commented 5 months ago

yes sure I'll highlight them more. It will not be displayed as a table like on the request part though

Any reason not to? Depending on the length of the longest header it might be not the best utilisation of the free space, however in situation with many similar headers is might be handy.

your first reason was the right one. Tables are hard to implement when considering things like scrolling. And as you said, for long headers it will more annoying so let's keep it simple