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 headers highlighting #31

Closed
savchenko opened this issue Apr 18, 2024 · 6 comments · Fixed by #56
Closed

Add headers highlighting #31

savchenko opened this issue Apr 18, 2024 · 6 comments · Fixed by #56

Comments

@savchenko
Copy link

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

@savchenko savchenko changed the title Add headers highlight Add headers highlighting Apr 18, 2024
@Julien-cpsn
Copy link
Owner

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

@Julien-cpsn
Copy link
Owner

@savchenko

@savchenko
Copy link
Author

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
Copy link
Owner

Julien-cpsn commented Apr 28, 2024

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
Copy link
Author

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
Copy link
Owner

Julien-cpsn commented Apr 28, 2024

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

@Julien-cpsn Julien-cpsn linked a pull request May 1, 2024 that will close this issue
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 a pull request may close this issue.

2 participants