Summary: | (minor)AntView status line shows "no description" | ||
---|---|---|---|
Product: | [Eclipse Project] Platform | Reporter: | Erich Gamma <erich_gamma> |
Component: | Ant | Assignee: | Darin Swanson <Darin_Swanson> |
Status: | VERIFIED FIXED | QA Contact: | |
Severity: | enhancement | ||
Priority: | P3 | Keywords: | ui |
Version: | 2.1 | ||
Target Milestone: | 2.1 RC2 | ||
Hardware: | PC | ||
OS: | Windows 2000 | ||
Whiteboard: |
Description
Erich Gamma
2003-02-21 17:33:24 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. 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. What do you think about making the string mirror the target definition in the XML? name=rebuild all depends=init,clean description=Rebuild all 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. 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. Verified. |