Skip to content

diagnostics/traffic: Improve Top talkers table in traffic#7979

Closed
Ashus wants to merge 2 commits intoopnsense:masterfrom
Ashus:improve_top_talkers
Closed

diagnostics/traffic: Improve Top talkers table in traffic#7979
Ashus wants to merge 2 commits intoopnsense:masterfrom
Ashus:improve_top_talkers

Conversation

@Ashus
Copy link
Copy Markdown

@Ashus Ashus commented Oct 18, 2024

  • Add bit/byte switch, current and max traffic is shown in selected units
  • Allow to open directly with Top talkers tab active with URL #toptalkers
  • Instead of last timestamp in ISO format show time in seconds an address was seen last
  • Highlight values over 0.5 MB/s current traffic and values over 10 MB totals, if any of those is highlighted, the row is highlighted as well
  • Mark rows where traffic only ever came in one direction as gray - unanswered
  • Ability to click a row to mark it tracked, easier to follow someone in a table where the top get sorted often
  • Add an action button to show sessions related to the IP
  • Fix for mobile view - table does not overflow and can be scrolled

* Add bit/byte switch, current and max traffic is shown in selected units
* Allow to open directly with Top talkers tab active with URL #toptalkers
* Instead of last timestamp in ISO format show time in seconds an address was seen last
* Highlight values over 0.5 MB/s current traffic and values over 10 MB totals, if any of those is highlighted, the row is highlighted as well
* Ability to click a row to mark it tracked, easier to follow someone in a table where the top get sorted often
* Add an action button to show sessions related to the IP
* Add first seen column with formatted time
@AdSchellevis AdSchellevis force-pushed the master branch 5 times, most recently from 6586a65 to 607e32a Compare December 3, 2024 15:53
@Ashus Ashus deleted the improve_top_talkers branch September 6, 2025 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants