Whole Tomato Software Forums
Whole Tomato Software Forums
Main Site | Profile | Register | Active Topics | Members | Search | FAQ
User name:
Password:
Save Password
Forgot your password?

 All Forums
 Visual Assist
 Feature Requests
 VA Outline suggestion
 New Topic  Reply to Topic
 Printer Friendly
Author Previous Topic Topic Next Topic  

znakeeye
Tomato Guru

379 Posts

Posted - Feb 18 2008 :  03:15:02 AM  Show Profile  Reply with Quote
I see this:

VA Outline
+ CMySpecialThumbnailPropertyPage methods
CMySpecialThumbnailPropertyPage::Method1
CMySpecialThumbnailPropertyPage::Method2
etc..

Now, my Ouline-window is some 200 pixels wide. I do not want to stretch it over the whole screen just to see the methods. I think "Class::" should be removed below "+ CMySpecialThumbnailPropertyPage methods".

A better approach would be to show this "Class::" in the tooltip!

accord
Whole Tomato Software

United Kingdom
3287 Posts

Posted - Feb 18 2008 :  05:47:17 AM  Show Profile  Reply with Quote
This suggested approach have an another benefit: now I can search in the header, but cannot in the cpp. With this improvement I will able to find function name in the cpp also.

You know, I am talking about the windows' default search: you select a node, and then start typing...
Go to Top of Page

feline
Whole Tomato Software

United Kingdom
18939 Posts

Posted - Feb 18 2008 :  11:40:30 AM  Show Profile  Reply with Quote
This runs into problems as soon as your file contains more than one class, or it contains "free" functions that are members of no class. Without the class name you cannot tell at a glance which is which.

The parent node tells you which class this function belongs to, but if you have a complex file, or a large number of functions, then you may not be able to quickly and easily tell which parent relates to the current item.

I understand the appeal, but do the benefits out way the costs?

accord is there a reason for searching for a function in VA Outline? This is what the alt-m list is designed for. VA Outline is designed to track your position in the file, and jump to and highlight the function the caret is currently inside.

zen is the art of being at one with the two'ness
Go to Top of Page

znakeeye
Tomato Guru

379 Posts

Posted - Feb 20 2008 :  05:17:36 AM  Show Profile  Reply with Quote
I see your point, but in my opinion the tooltip is sufficient for showing the class name. When dealing with long class names the whole VA window loses its power.

When I have a decent size of the VA Outline pane, I often end up with information such as this:

CCaseThumbnailPropertyPage methods
| CCaseThumbnailPropertyPage::A|
| CCaseThumbnailPropertyPage::C|
| CCaseThumbnailPropertyPage::D|
| CCaseThumbnailPropertyPage::E|

There should at least be an option to turn this off. In our projects we seldom have multiple classes in one file.
Go to Top of Page

feline
Whole Tomato Software

United Kingdom
18939 Posts

Posted - Feb 20 2008 :  08:25:19 AM  Show Profile  Reply with Quote
I have put in a feature request for this. What you are seeing is clearly not that helpful:

case=13240

zen is the art of being at one with the two'ness
Go to Top of Page

accord
Whole Tomato Software

United Kingdom
3287 Posts

Posted - Feb 24 2008 :  2:51:21 PM  Show Profile  Reply with Quote
quote:
I have put in a feature request for this.

So, I don't have to answer to the "why" question

This wasn't very important for me, but case=13240 will work also for me I think it is a good idea to hide sometimes useless information. This may (or may not) help for Esone also, he can see more of his parameter list, so his bird can fly higher : http://forum.wholetomato.com/forum/topic.asp?TOPIC_ID=7178

So everybody is much more happy now

Edited by - accord on Feb 24 2008 3:04:54 PM
Go to Top of Page

znakeeye
Tomato Guru

379 Posts

Posted - Jul 08 2008 :  11:43:53 AM  Show Profile  Reply with Quote
No, this is... bad UI thinking!

There is a reason you don't specify the class name for each member function in a header file. It's simply not necessary. Just like in Explorer, the folder "Visual Assist X" is not named "C:\\Program Files\\Visual Assist X", because it's not necessary!

Similarly, that piece of information is not necessary in the VA Outline window. Regarding "multiple classes in a header file", you should add an extra feature where you select the class in a drop down (or similar) to explicitly work with that perticular class. It makes no sense to work with multiple classes. Actually, this "drop down" thingy would be great!

I win ;)
Please fix/add this in the next release.
Go to Top of Page

feline
Whole Tomato Software

United Kingdom
18939 Posts

Posted - Jul 08 2008 :  4:03:08 PM  Show Profile  Reply with Quote
So if you are working in a file that contains multiple classes you cannot use VA Outline to drag and drop items between the classes?

What about header files that contain things other than the class? #include lines, macro's, template functions, comment blocks? These things need to be clearly distinguished from the class.

zen is the art of being at one with the two'ness
Go to Top of Page

znakeeye
Tomato Guru

379 Posts

Posted - Jul 13 2008 :  12:35:14 PM  Show Profile  Reply with Quote
Or you could simply add these lines in your code:

if (AfxGetApp()->GetProfileInt("VANet9", "OutlineShowUnnecessaryClassName", FALSE))
{
strItem.Append(strClassName);
strItem.Append("::");
}

Go to Top of Page

feline
Whole Tomato Software

United Kingdom
18939 Posts

Posted - Jul 14 2008 :  09:09:01 AM  Show Profile  Reply with Quote
If you are suggesting adding the class name to the functions, instead of having the class as a parent node, then people will immediately complain that this makes the items to long, and for a long class name they can no longer read the function names.

zen is the art of being at one with the two'ness
Go to Top of Page

znakeeye
Tomato Guru

379 Posts

Posted - Jul 15 2008 :  08:50:03 AM  Show Profile  Reply with Quote
I'm suggesting the exact opposite...

The class name should not be added for its sub-items! Example:
- CMyClass methods
CMyClass::Foo()
CMyClass::Bar()

Redundant information inc. This is what I've been asking for:

- CMyClass methods
Foo()
Bar()
Go to Top of Page

feline
Whole Tomato Software

United Kingdom
18939 Posts

Posted - Jul 15 2008 :  3:20:51 PM  Show Profile  Reply with Quote
This is on our list of things to do:

case=13240

zen is the art of being at one with the two'ness
Go to Top of Page

dgerman
New Member

2 Posts

Posted - Aug 04 2008 :  1:32:10 PM  Show Profile  Reply with Quote
Would it be possible to print the outline or save to an image file? I like Visual StudioG??s class diagram that works only with .net code. Could you create something similar for visual c++?
Go to Top of Page

feline
Whole Tomato Software

United Kingdom
18939 Posts

Posted - Aug 06 2008 :  09:55:53 AM  Show Profile  Reply with Quote
We are considering a feature to let you copy or print the content of VA Outline as text:

case=11169

What is this class diagram and how do I trigger it? I am not seeing an obvious menu option in VS2005.

zen is the art of being at one with the two'ness
Go to Top of Page

dgerman
New Member

2 Posts

Posted - Aug 07 2008 :  08:29:36 AM  Show Profile  Reply with Quote
quote:
Originally posted by feline

We are considering a feature to let you copy or print the content of VA Outline as text:

case=11169

What is this class diagram and how do I trigger it? I am not seeing an obvious menu option in VS2005.




If for example in a .net c# project you can right click an item in the solution explorer you get as one of the menu items a "View Class Diagram"

Thanks
Go to Top of Page

feline
Whole Tomato Software

United Kingdom
18939 Posts

Posted - Aug 08 2008 :  08:56:16 AM  Show Profile  Reply with Quote
What is the point of this class diagram?

I have triggered this on a couple of different C# files, and I am just getting one "blob" per class, and a list of the class members in the bottom section of the screen when I click on the "blob".

This does not actually seem to do anything useful, or offer me anything useful.

VA Outline already shows me the class members as children of the class.

zen is the art of being at one with the two'ness
Go to Top of Page

znakeeye
Tomato Guru

379 Posts

Posted - Nov 25 2008 :  05:59:17 AM  Show Profile  Reply with Quote
Will this be added in 10.5? It's a shame VA Outline becomes almost unusable due to this :(
Go to Top of Page

feline
Whole Tomato Software

United Kingdom
18939 Posts

Posted - Nov 25 2008 :  09:05:37 AM  Show Profile  Reply with Quote
Which feature are you asking about? This thread discusses several.

The new beta VA 1707 contains some changes to VA Outline.

zen is the art of being at one with the two'ness
Go to Top of Page

support
Whole Tomato Software

5566 Posts

Posted - Dec 01 2008 :  07:33:46 AM  Show Profile  Reply with Quote
case=13240 is implemented in build 1707
Go to Top of Page

sean
Whole Tomato Software

USA
2817 Posts

Posted - Apr 07 2014 :  3:29:26 PM  Show Profile  Reply with Quote
case=11169 is implemented in build 2031
Go to Top of Page
  Previous Topic Topic Next Topic  
 New Topic  Reply to Topic
 Printer Friendly
Jump To:
© 2023 Whole Tomato Software, LLC Go To Top Of Page
Snitz Forums 2000