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

Remember the sort order of lists across sessions #507

Closed
1 task done
Digitalone1 opened this issue Apr 14, 2024 · 3 comments
Closed
1 task done

Remember the sort order of lists across sessions #507

Digitalone1 opened this issue Apr 14, 2024 · 3 comments
Labels
enhancement New feature, request, or improvement

Comments

@Digitalone1
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues.

Describe the solution you'd like

Maybe the sort order of lists can be saved and reapplied at the next restart, fort both overview and inspect tabs.

Is your feature request related to a problem?

Every time I start the app, I should reset the sort order. Maybe this is not relevant for users that prefer the default state, but it's quite annoying for others that prefer a different order.

Maybe they could be saved to the config file on shutdown and reloaded at startup.

This is not a big issue, but the usability could be improved after adding this feature.

Anyway, I didn't say it before, but thanks for this application, it's very helpful. There are other tools on Linux that does quite the same thing, but at least sniffnet is more user friendly.

@GyulyVGC
Copy link
Owner

Adding this is extremely trivial since Sniffnet already uses a config file for the parameters included in the settings.

My only concern is whether it's actually meaningful or not.
I'm not sure this kind of stuff is usually saved into configs, and maybe it could be non-intuitive for some users.
I can think of a case where a user changes the sort order just out of curiosity and then reopens the app expecting to see most recent data without noticing that the sort order is still that set last time.

I feel seeing most recent data is what majority of the people are interested in.
Why do you prefer a different sorting strategy? Maybe getting your point of view can help.

Anyway, I didn't say it before, but thanks for this application, it's very helpful. There are other tools on Linux that does quite the same thing, but at least sniffnet is more user friendly.

Thanks so much 🙏
I'm also thankful to you for all the opened issues, they are indeed useful for me / the project.

@GyulyVGC GyulyVGC added the enhancement New feature, request, or improvement label Apr 14, 2024
@Digitalone1
Copy link
Contributor Author

My only concern is whether it's actually meaningful or not. I'm not sure this kind of stuff is usually saved into configs, and maybe it could be non-intuitive for some users. I can think of a case where a user changes the sort order just out of curiosity and then reopens the app expecting to see most recent data without noticing that the sort order is still that set last time.

I feel seeing most recent data is what majority of the people are interested in. Why do you prefer a different sorting strategy? Maybe getting your point of view can help.

To see which connection is using more data. I think the user expects to see the sort order they set previously. In example, if I set a specific order in Thunderbird to get mails sorted in the way I want, at the next startup the order does not change.

@GyulyVGC
Copy link
Owner

After more carefully evaluating this request I've come to the conclusion that I prefer leaving things as they're now.

Despite adding this configuration is straightforward, the running pages of the app (Overview, Inspect, Notifications) are stateless by design: the idea is that when a new capture is restarted, it begins from a completely clean state.

Adding states for the sort orders would make this more ambiguous, since other parameters'd be reset anyway:

  • data representation unit
  • favorite hosts
  • search filters in Inspect page
  • notifications list

The parameters with a state are included in the settings of the app, and none of them is relative to the analysis itself (except for the MMDB files, but they are more time consuming to select than a simple click on the sort arrows, and they are not directly visible in the running pages).

@GyulyVGC GyulyVGC closed this as not planned Won't fix, can't repro, duplicate, stale Apr 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature, request, or improvement
Projects
None yet
Development

No branches or pull requests

2 participants