Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 322110 - The "review" field is not rendered correctly
Summary: The "review" field is not rendered correctly
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.5   Edit
Assignee: Frank Becker CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-09 04:06 EDT by Ankur Sharma CLA
Modified: 2011-01-13 23:04 EST (History)
1 user (show)

See Also:

Attachments
snapshot (6.25 KB, image/jpeg)
2010-08-09 04:06 EDT, Ankur Sharma CLA 		no flags Details
commited patch (12.09 KB, patch)
2010-08-14 14:59 EDT, Frank Becker CLA 		no flags Details | Diff
mylyn/context/zip (4.38 KB, application/octet-stream)
2010-08-14 14:59 EDT, Frank Becker CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ankur Sharma CLA 2010-08-09 04:06:41 EDT 
Created attachment 176130 [details]
snapshot

Repository: Eclipse.org

The fields "review" and "pmc_approved" are not rendered properly (see snapshot).

Also, it will be good enhancement if these fields can support the auto-completion like 'Add CC' field.
Comment 1 Frank Becker CLA 2010-08-14 14:59:42 EDT 
Created attachment 176603 [details]
commited patch
Comment 2 Frank Becker CLA 2010-08-14 14:59:44 EDT 
Created attachment 176604 [details]
mylyn/context/zip
Comment 3 Frank Becker CLA 2010-08-14 15:04:33 EDT 
(In reply to comment #0)
> Created an attachment (id=176130)
> snapshot
> 
> Repository: Eclipse.org
> 
> The fields "review" and "pmc_approved" are not rendered properly (see snapshot).

I create the patch on an Mac so please verify.  
> 
> Also, it will be good enhancement if these fields can support the
> auto-completion like 'Add CC' field.

The enhancement is also included.
Comment 4 Ankur Sharma CLA 2010-08-15 00:49:50 EDT 
Thanks for a quick resolution