Bugzilla – Full Text Bug Listing
| Summary: | Git status commands need user readable names and tooltips | ||
|---|---|---|---|
| Product: | [ECD] Orion | Reporter: | John Arthorne <john.arthorne> |
| Component: | Client | Assignee: | libing wang <libingw> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | simon_kaegi, susan |
| Version: | unspecified | ||
| Target Milestone: | 0.3 RC2 | ||
| Hardware: | PC | ||
| OS: | Windows 7 | ||
| Whiteboard: | |||
Several things going on here. In bug 358162, we are addressing that we have never used the tooltip anywhere in the UI. We are going to set the tooltip in the command anchor tag that contains text or image. However, for image commands, we also set the tooltip for the image to the command name. This is so that we can explain the icons. So the name will show in the icon case, the tooltip will show in the text case. Since general use of tooltip is covered in bug 358162, let's target this bug to the stage all and unstage all commands, which need proper names. The name should be set to "Unstage All" and a real tooltip defined. Such as: var unstageAllCommand = new mCommands.Command({ name: "Unstage All", tooltip: "Unstage all staged files", (One could argue that the tooltip is not needed/doesn't add much). Libing, could you check all your git status commands and ensure that: - the "name" property is set to a user readable name. This name will appear when the command is contributed as a text link, and will also appear as the hover when the icon is used. - the "tooltip" property is set to a short descriptive tooltip. This appears as the hover when the name is already shown in a text link. Is this done? yes. Fixed with fde48456846a7cacb6844b429453bcd4835a3743. |
Commands have a name and an optional tooltip. However, in the tooltip hover we are always showing the command name rather than the tooltip attribute. The tooltip attribute doesn't ever seem to be used. This shows up in the "Stage all" and "Unstage all" commands, which look like this: var unstageAllCommand = new mCommands.Command({ name: "unstageAll", tooltip: "Unstage all", In the UI, the tooltip hover shows the goofy "unstageAll" rather than the actual tooltip, "Unstage all".