Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 359807 - Last command not displayed in status bar
Summary: Last command not displayed in status bar
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.4   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.5 M3   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard: sr142
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-04 05:18 EDT by Julien Saulou CLA
Modified: 2011-10-09 04:55 EDT (History)
0 users

See Also:


Attachments
Project that reproduces thie problem (29.72 KB, application/octet-stream)
2011-10-04 05:20 EDT, Julien Saulou CLA
no flags Details
Proposed patch for v14_Maintenance branch (1.80 KB, patch)
2011-10-08 01:53 EDT, Ivan Furnadjiev CLA
ruediger.herrmann: review+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Julien Saulou CLA 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 CLA 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 CLA 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 CLA 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 CLA 2011-10-08 01:53:57 EDT
Created attachment 204786 [details]
Proposed patch for v14_Maintenance branch
Comment 5 Ivan Furnadjiev CLA 2011-10-09 04:55:53 EDT
Applied patch to v14_Maintenance.