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

Bug 359807

Summary:

Last command not displayed in status bar

Product:

[RT] RAP

Reporter:

Julien Saulou <julien.saulou>

Component:

RWT

Assignee:

Project Inbox <rap-inbox>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

Version:

1.4

Target Milestone:

1.5 M3

Hardware:

All

OS:

All

Whiteboard:

sr142

Attachments:

Description	Flags
Project that reproduces thie problem	none
Proposed patch for v14_Maintenance branch	ruediger.herrmann: review+

Description Julien Saulou

2011-10-04 05:18:10 EDT

Hi,

We are getting a problem using commands in the status bar.
We're placing multiple commands and the last one is not displayed on the first connection.
This does not seep to happen when we don't use FORCE_TEXT for the commands, so this may be related to some kind on text size determination issue.

The bug that looks quite similar (bug #339334).

Comment 1 Julien Saulou

2011-10-04 05:20:18 EDT

Created attachment 204498 [details]
Project that reproduces thie problem

Simple project based on the RAP mail demo that reproduces the problem.

Comment 2 Ivan Furnadjiev

2011-10-05 06:00:27 EDT

Thanks for the snippet. I can reproduce the issue with your test project. The problem is more complicated than the bug 339334. Needs more investigation.

Comment 3 Ivan Furnadjiev

2011-10-08 01:47:52 EDT

The issue is the following. Status line toolbar is created packed and it's size does not change afterwards. Text size determination is using an internal flag packed to re-pack the packed controls. This flag was cleared in Control#setBounds. As Control#setBounds is called from Control#setLocation (without changing the packed size), this flag was wrongly cleared. The fix is to clear the packed flag in Control#setBounds *only* if the control size changes. Changes are in CVS HEAD.

Comment 4 Ivan Furnadjiev

2011-10-08 01:53:57 EDT

Created attachment 204786 [details]
Proposed patch for v14_Maintenance branch

Comment 5 Ivan Furnadjiev

2011-10-09 04:55:53 EDT

Applied patch to v14_Maintenance.