Bug 32521 - (minor)AntView status line shows "no description"
Summary: (minor)AntView status line shows "no description"
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Ant (show other bugs)
Version: 2.1   Edit
Hardware: PC Windows 2000
: P3 enhancement (vote)
Target Milestone: 2.1 RC2   Edit
Assignee: Darin Swanson CLA
QA Contact:
URL:
Whiteboard:
Keywords: ui
Depends on:
Blocks:
 
Reported: 2003-02-21 17:33 EST by Erich Gamma CLA
Modified: 2003-03-04 09:18 EST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Erich Gamma CLA 2003-02-21 17:33:24 EST
The status line of the AntView shows the following text:
Name: gather.logs Dependencies: init Description: (no description)

* it is difficult to distinguish the fixed text from the variable (interesting 
text). Suggest to minimize the fixed test, e.g. remove "Name:"

* when there is no Description then omit the field  alltogether showing "(no 
description)" has only minimal value. 

Here is a proposal:
gather.logs, depends: init, "its description"
Comment 1 Jared Burns CLA 2003-03-03 12:40:07 EST
Separating the depends from the description with commas would be confusing
for targets with multiple dependencies. For example:
gather.logs, depends: init, do it, another target, a description

We need to keep the "depends:" (agree we should shorten it from
"dependencies") and "description" labels for this reason. I added the "name"
label only for consistency but we can remove it if you think it takes up too
much space.

I also agree that we can lose the "description: (no description)" text.
Comment 2 Erich Gamma CLA 2003-03-03 12:46:48 EST
An alternative would be to show the description separated by a -, e.g.,
   gather.logs, depends: init, do it, another target - a description

However, I have no strong opinion on this one, but I still suggest to remove 
the Name: field.

Comment 3 Jared Burns CLA 2003-03-03 12:49:07 EST
What do you think about making the string mirror the target definition in the
XML?
name=rebuild all depends=init,clean description=Rebuild all
Comment 4 Erich Gamma CLA 2003-03-03 13:13:08 EST
this would make sense, but I find that without quotes it is difficult to parse:

name=rebuild all depends=...

vs

name="rebuild all" depends=...

However, adding quotes makes the line more noisy. 
Comment 5 Jared Burns CLA 2003-03-03 16:57:51 EST
Went with the XML-style naming which makes it easier to parse the status line
visually and removed the (no description) text.

Please verify, DarinS.
Comment 6 Darin Swanson CLA 2003-03-04 09:18:21 EST
Verified.