This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 272092 - [editor] streamline the task editor look-and-feel
Summary: [editor] streamline the task editor look-and-feel
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 3.2   Edit
Assignee: Mik Kersten CLA
QA Contact:
URL:
Whiteboard:
Keywords: plan
Depends on: 211096 277970 277972 277984 278133 278269
Blocks:
  Show dependency tree
 
Reported: 2009-04-13 19:46 EDT by Mik Kersten CLA
Modified: 2009-06-11 11:07 EDT (History)
4 users (show)

See Also:


Attachments
patch for notes overlay (4.17 KB, patch)
2009-05-28 19:42 EDT, Shawn Minto CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mik Kersten CLA 2009-04-13 19:46:23 EDT
The task editor gets a lot of a Mylyn user's interaction, so we should take a pass at improving the visual design and layout.
Comment 1 Mik Kersten CLA 2009-05-27 19:12:59 EDT
Shawn: Could you move "Personal Planning" to below "Attributes"?
Comment 2 Mik Kersten CLA 2009-05-28 00:06:00 EDT
Misc nits (for Shawn I think):
* Rename all instances of "Local" to "Private"
* Get rid of "Notes"...
* "Planning" (tab) -> "Private"
* Fix width problem with "Private" section
* Move "Private" section to just below "Attributes" section
Comment 3 Steffen Pingel CLA 2009-05-28 00:28:29 EDT
* The Planning section should be expanded if it has notes or there should be an indication in the section header, e.g. the first sentence of the notes.
Comment 4 Steffen Pingel CLA 2009-05-28 00:37:26 EDT
Shawn, I made the notes expand to the size of the display if there is a lot of text (consistent with the behavior of the description). We could still consider adding a maximize icon to the section header.
Comment 5 Shawn Minto CLA 2009-05-28 15:02:22 EDT
(In reply to comment #2)
> Misc nits (for Shawn I think):
> * Rename all instances of "Local" to "Private"
Mik, did you want to take care of this one?

> * Get rid of "Notes"...
Done.  I still want to add an overlay if there are no notes so users know what this field is for.

> * "Planning" (tab) -> "Private"
Done.
> * Fix width problem with "Private" section
Steffen took care of this.
> * Move "Private" section to just below "Attributes" section
Done.
(In reply to comment #3)
> * The Planning section should be expanded if it has notes or there should be an
> indication in the section header, e.g. the first sentence of the notes.
I have made this auto-expand for now.  Lets see what we think.

I have also added the save button to the left header to be consistent with the rest of the task editors.
Comment 6 Shawn Minto CLA 2009-05-28 19:42:17 EDT
Created attachment 137582 [details]
patch for notes overlay

Here is a quick patch for making a text overlay on the notes field when there is not text to inform users of what the field is for.  The weird thing about this is that there are multiple viewers on the RichTextEditor, so there is some duplication.  Let me know what you think before I commit this change.
Comment 7 Mik Kersten CLA 2009-05-28 21:34:32 EDT
Steffen: Could you review?
Comment 8 Steffen Pingel CLA 2009-05-28 23:50:23 EDT
Shawn, have you tried calling editor.getViewer()? That should always return the current viewer.
Comment 9 Mik Kersten CLA 2009-05-29 00:08:01 EDT
Steffen: Could you add the "Attach File" icon to the Attachments section?
Comment 10 Steffen Pingel CLA 2009-05-29 00:34:58 EDT
(In reply to comment #9)
> Steffen: Could you add the "Attach File" icon to the Attachments section?

Done.
Comment 11 Steffen Pingel CLA 2009-05-29 02:12:21 EDT
Shawn, it might make sense to enable the repository link in the header of Local task editors now that the Local repository has a settings page.
Comment 12 Shawn Minto CLA 2009-05-29 11:15:32 EDT
I have added the repository link and have committed my change for the notes (thanks for the pointer steffen).  Let me know what you think.
Comment 13 Steffen Pingel CLA 2009-06-01 17:28:59 EDT
I have reverted the right-alignment of the date labels in the header. While it works great on small monitors I found that it is easy to miss the information in the right column since all text is left aligned and the editor only has a single column.
Comment 14 Steffen Pingel CLA 2009-06-01 19:28:36 EDT
All subtasks have been completed.
Comment 15 Mik Kersten CLA 2009-06-02 16:09:58 EDT
I'm not sure I follow.  Was the main problem that on large monitors the dates were easy to miss because the user was not likely to scan the right-hand side of the editor under the toolbar?  I could see how that's an issue with the Attributes section open, since that section does not fill available space.  But with the Attributes section closed, I thought that the idea was that we would put key attributes on the right side of each collapsed section.

Also, with this change that nice separation we had of summary info and dates is gone.
Comment 16 Steffen Pingel CLA 2009-06-02 20:55:55 EDT
I found that I only scan the first 600px of the left column in the task editor when looking at the anything other than comments and description so I repeatedly thought tasks were missing date information. We have also had reports from users who never noticed the icons on the top right of section headers. This might be different if all sections of the task editor filled the horizontal space but with the current design I don't see any benefit in moving the dates or other attribute information to the right.

The biggest problem I noticed on small monitors is that when you end up with a horizontal scroll bar the information in the right column is not visible anymore at all without scrolling.
Comment 17 Steffen Pingel CLA 2009-06-10 21:32:25 EDT
As per discussion with Mik we'll keep the left alignment of controls for now to avoid lots of blank space in the center of the header. We may still consider a multi-column layout in the future.
Comment 18 Mik Kersten CLA 2009-06-11 11:07:50 EDT
Another task bites the dust :)  I'm really glad we were able to put all these cycles into the task editor for 3.2.