Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 349510

Summary:

add maximize button to private notes section

Product:

z_Archived

Reporter:

Sam Davis <sam.davis>

Component:

Mylyn

Assignee:

Sam Davis <sam.davis>

Status:

RESOLVED FIXED

QA Contact:

Severity:

enhancement

Priority:

CC:

steffen.pingel

Version:

unspecified

Keywords:

bugday, contributed, helpwanted

Target Milestone:

3.7

Hardware:

OS:

Windows 7

Whiteboard:

Bug Depends on:

Bug Blocks:

158921

Attachments:

Description	Flags
patch	none
mylyn/context/zip	none
updated	none
mylyn/context/zip	none
patch	steffen.pingel: iplog+

Description Sam Davis

2011-06-15 19:10:55 EDT

It would be great to have a maximize button for the private notes section. Current workaround is to fill it with text and close and reopen the task. That makes it bigger but not maximized.

Comment 1 Steffen Pingel

2011-07-26 14:10:47 EDT

Agreed. I am marking this as helpwanted to indicate that this would be a great enhancement to resolve through a contribution. Note bug 288171 which should provide the necessary API to make this fairly easy to implement.

Comment 2 Sam Davis

2011-12-21 20:48:08 EST

Created attachment 208715 [details]
patch

Comment 3 Sam Davis

2011-12-21 20:48:11 EST

Created attachment 208716 [details]
mylyn/context/zip

Comment 4 Steffen Pingel

2011-12-23 13:14:56 EST

Thanks for the patch. Looks good. Only minor nit I noticed is that the section is also enlarged when not expanded resulting in a lot of blank space in the task editor. Could fix that?

Comment 5 Sam Davis

2012-01-03 16:59:12 EST

Created attachment 208969 [details]
updated

Comment 6 Sam Davis

2012-01-03 16:59:15 EST

Created attachment 208970 [details]
mylyn/context/zip

Comment 7 Steffen Pingel

2012-01-05 14:57:49 EST

Sam, can you elaborate on the changes? It's not obvious to me what the purpose of layoutControl is. Have you tried using CommonFormUtil.setExpanded() to expand the section? That should trigger the expansion event.

Comment 8 Sam Davis

2012-01-05 16:24:53 EST

Created attachment 209105 [details]
patch

layoutControl has a GridLayout, which is needed for the maximize action to work; setting a GridData on the section client would have no effect.

Thanks for the tip about CFU; here's an updated patch that gets rid of the unnecessary listener.

Comment 9 Steffen Pingel

2012-01-05 18:49:12 EST

Great! I have applied the patch to master.