Stephen Fisher wrote:
> I have committed an initial version of a content listing/saving feature 
> for the HTTP protocol.  
First of all, thanks for this - it's a feature long awaited IMO.
> I would appreciate if anyone could try it out 
> and give feedback on the implementation and if they can think of a 
> better top-level menu to put it under (View perhaps?). 
>   
Basically, the View menu is about *how* things are displayed, and not to 
display "new" things - so this is not the place to go.

Analyze or Statistics is the place to go - IMO it might better be placed 
in Analyze as this feature is not really about Statistics. However, both 
menus have become a bit interchangeable over time.

Some things I've noticed:

- this announcement should have gone to the developer list first (most 
developer related discussion will go to the users list now)
- the buttons don't have a tooltip (should be easy to add)
- when selecting a row, the packet list should jump to that packet (as 
other similar dialogs do it)
- save fails if the "suggested" filename contains bad characters (e.g. 
question mark), which often happens. Unfortunately, I don't know a good 
character encapsulation for this.
- no "Save All" button (to save all files in one rush)
- not mentioned in the User's Guide (that's ok as this feature is 
currently experimental) - but shall be added *before* the next release
- no "Help" button to point to the none existing User's Guide section ;-)

Anyway, this is a really good start of this topic - now it's about 
getting it "bullet proof" ;-)

Regards, ULFL
_______________________________________________
Wireshark-users mailing list
[email protected]
http://www.wireshark.org/mailman/listinfo/wireshark-users

Reply via email to