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.

Sign up for GitHub

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

Jump to bottom

Make "text viewer" the default tab in the packet inspector. #452

Closed
rj00a opened this issue Aug 8, 2023 · 1 comment · Fixed by #459
Closed

Make "text viewer" the default tab in the packet inspector. #452

rj00a opened this issue Aug 8, 2023 · 1 comment · Fixed by #459
Assignees
@ShivangRawat30
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@rj00a
Copy link
Member

rj00a commented Aug 8, 2023 •
edited
Loading

Describe the problem related to your feature request.

Clicking on a packet in the packet inspector shows the "hex viewer" by default. However, the "text viewer" is usually what you want.

What solution would you like?

Make the text viewer tab the default. It should also be placed left of the hex viewer.
@rj00a rj00a added enhancement New feature or request good first issue Good for newcomers labels Aug 8, 2023
@ShivangRawat30
Copy link

Hey I am currently learning rust, as it is a good first issue can you please assign this to me.

@AviiNL AviiNL mentioned this issue Aug 9, 2023
Merged
@rj00a rj00a closed this as completed in #459 Aug 9, 2023
rj00a pushed a commit that referenced this issue Aug 9, 2023
452 swap hex text view (#459)
6c730ab 
# Objective

Fixes #452 

guess i was looking at the wrong tab when creating my branch lol
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ShivangRawat30 ShivangRawat30

Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

452 swap hex text view AviiNL/valence
2 participants
@rj00a @ShivangRawat30