Wireshark-dev: Re: [Wireshark-dev] [PATCH] New menu items to copy packet data
From: Ulf Lamping <ulf.lamping@xxxxxx>
Date: Sun, 28 Jan 2007 05:39:41 +0100
sfisher@xxxxxxxxxxxxx wrote:
Log: From Douglas Pratley:This is a patch adding items to the context menus for the packet listand packet details panes for copying packet data to the clipboard.New menu item on packet list pane:Copy (sub menu) -> Summary (Text) Copies fields displayed in the packet list, tab separated Summary (CSV) Copies fields displayed in the packet list, comma separated Hex and Text Copies data for the selected frame in hex-editor style (similar to Copy\All Information in packet bytes pane) Text Only Copies data for the selected frame, dropping non-printable characters other than line breaks (similar to Copy\Text Only in packet bytes pane) Hex Columns As Hex and Text but without text Hex Stream Copies data for the selected frame as an unpunctuated list of hex values Binary Stream Copies raw packet bytes to the clipboard as MIME type application/octet-stream
Nice work! As usual some GUI nitpicking from me ;-)Comparing the context menus (packet list, details, bytes), they have no common way they are named (especially in combination with the existing menu items), it's looking a bit weird now.
In addition, I had to try some things out before I get the actual meaning. So a new layout proposal of the corresponding menu items: 1.) Packet List / Copy item: Summary (Text) Summary (CSV) ------------------ Bytes (Offset Hex Text) Bytes (Offset Hex) Bytes (Printable Text Only) ------------------ Bytes (Hex Stream) Bytes (Binary Stream) 2.) Packet Details / Copy item (only one Copy item in this menu): Description ------------------ Bytes (Offset Hex Text) Bytes (Offset Hex) Bytes (Printable Text Only) ------------------ Bytes (Hex Stream) Bytes (Binary Stream) 3.) Packet Bytes / Copy item:The usability of this menu is completely odd, as you select things you don't want.
With the new Details menu the dialog menu is completely obsolete - after some cleanup. Namely moving the "Export Selected Packet Bytes" item to Packet Details menu (shouldn't be too difficult?!?). We might also want to remove this export feature completely, as I guess that the "Copy" mechanism is what users really need ...
What do you think about all the name changes?BTW: Is the "Printable Text" a good idea? Wouldn't the "common" replacement of unprintable chars by dots be more common / helpful? Or is there even a need for both?
Again, your work was already overdue in my eyes, thanks for doing it! Regards, ULFL
- Prev by Date: Re: [Wireshark-dev] [PATCH] New menu items to copy packet data
- Next by Date: Re: [Wireshark-dev] Dissector for Cisco ITP packet logging facility
- Previous by thread: Re: [Wireshark-dev] [PATCH] New menu items to copy packet data
- Next by thread: Re: [Wireshark-dev] [PATCH] New menu items to copy packet data
- Index(es):
- Get Wireshark
- Download
- Code of Conduct