Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 351316 - [Button] SWT.ARROW exists but is unsupported
Summary: [Button] SWT.ARROW exists but is unsupported
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: unspecified   Edit
Hardware: PC All
: P4 normal (vote)
Target Milestone: 1.5 M7   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-06 08:23 EDT by Christian Radspieler CLA
Modified: 2012-04-24 04:31 EDT (History)
3 users (show)

See Also:

Attachments
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 Christian Radspieler CLA 2011-07-06 08:23:28 EDT 
Build Identifier: 1.4.0.20110614-2335

The image stays empty. I would expect that there is an arrow down in the button, if the style is SWT.ARROW | SWT.DOWN.

Reproducible: Always

Steps to Reproduce:
1. Create a button with style SWT.ARROW | SWT.DOWN
Comment 1 Andreas Blochberger CLA 2011-07-07 11:03:49 EDT 
I don't think this is an enhancement, because in a single source environment, this style does not work on RAP, but it works on RCP.
Though there is a workaround: not using this flag, but setting an image
Comment 2 Ralf Sternberg CLA 2011-07-07 17:40:40 EDT 
I'm sorry, I missed that the style flag ARROW is already available in RAP.  However, arrow buttons are not yet supported by the Button implementation. I don't remember how the style flag slipped in. Now it seems we'll have to support it ;-)
Comment 3 Andreas Blochberger CLA 2011-07-08 03:52:38 EDT 
Most of the support for SWT.ARROW is already part of the RWT Button class. The only thing that is missing, is the drawing of an image.

The win32 implementation of SWT Button uses the flag OWNERDRAW to accomplish that. In RWT an image might be a solution.
Comment 4 Ralf Sternberg CLA 2011-07-17 06:12:41 EDT 
Andreas, if you already have an idea how to solve this issue, would you like to provide a patch?
Comment 5 Tim Buschtoens CLA 2011-08-01 06:46:57 EDT 
Since a simple workaround exists we regard this bug as low priority.
Comment 6 Ivan Furnadjiev CLA 2012-04-24 04:31:53 EDT 
Fixed in CVS HEAD.